Fossil SCM
Use signed char for two json timeline functions that may return -1.
Commit
c42dee8634ba9961c4d03a9f43989a4a433831e1a22bcf34b9af279c08b98480
Parent
fd288be35672136…
1 file changed
+3
-3
+3
-3
| --- src/json_timeline.c | ||
| +++ src/json_timeline.c | ||
| @@ -143,12 +143,12 @@ | ||
| 143 | 143 | ** property is set to the tag/branch name found in the request. |
| 144 | 144 | ** |
| 145 | 145 | ** Only one of "tag" or "branch" modes will work at a time, and if |
| 146 | 146 | ** both are specified, which one takes precedence is unspecified. |
| 147 | 147 | */ |
| 148 | -static char json_timeline_add_tag_branch_clause(Blob *pSql, | |
| 149 | - cson_object * pPayload){ | |
| 148 | +static signed char json_timeline_add_tag_branch_clause(Blob *pSql, | |
| 149 | + cson_object * pPayload){ | |
| 150 | 150 | char const * zTag = NULL; |
| 151 | 151 | char const * zBranch = NULL; |
| 152 | 152 | char const * zMiOnly = NULL; |
| 153 | 153 | char const * zUnhide = NULL; |
| 154 | 154 | int tagid = 0; |
| @@ -220,11 +220,11 @@ | ||
| 220 | 220 | ** applies one of them. |
| 221 | 221 | ** |
| 222 | 222 | ** Returns -1 if it adds a "before" clause, 1 if it adds |
| 223 | 223 | ** an "after" clause, and 0 if adds only an order-by clause. |
| 224 | 224 | */ |
| 225 | -static char json_timeline_add_time_clause(Blob *pSql){ | |
| 225 | +static signed char json_timeline_add_time_clause(Blob *pSql){ | |
| 226 | 226 | char const * zAfter = NULL; |
| 227 | 227 | char const * zBefore = NULL; |
| 228 | 228 | int rc = 0; |
| 229 | 229 | zAfter = json_find_option_cstr("after",NULL,"a"); |
| 230 | 230 | zBefore = zAfter ? NULL : json_find_option_cstr("before",NULL,"b"); |
| 231 | 231 |
| --- src/json_timeline.c | |
| +++ src/json_timeline.c | |
| @@ -143,12 +143,12 @@ | |
| 143 | ** property is set to the tag/branch name found in the request. |
| 144 | ** |
| 145 | ** Only one of "tag" or "branch" modes will work at a time, and if |
| 146 | ** both are specified, which one takes precedence is unspecified. |
| 147 | */ |
| 148 | static char json_timeline_add_tag_branch_clause(Blob *pSql, |
| 149 | cson_object * pPayload){ |
| 150 | char const * zTag = NULL; |
| 151 | char const * zBranch = NULL; |
| 152 | char const * zMiOnly = NULL; |
| 153 | char const * zUnhide = NULL; |
| 154 | int tagid = 0; |
| @@ -220,11 +220,11 @@ | |
| 220 | ** applies one of them. |
| 221 | ** |
| 222 | ** Returns -1 if it adds a "before" clause, 1 if it adds |
| 223 | ** an "after" clause, and 0 if adds only an order-by clause. |
| 224 | */ |
| 225 | static char json_timeline_add_time_clause(Blob *pSql){ |
| 226 | char const * zAfter = NULL; |
| 227 | char const * zBefore = NULL; |
| 228 | int rc = 0; |
| 229 | zAfter = json_find_option_cstr("after",NULL,"a"); |
| 230 | zBefore = zAfter ? NULL : json_find_option_cstr("before",NULL,"b"); |
| 231 |
| --- src/json_timeline.c | |
| +++ src/json_timeline.c | |
| @@ -143,12 +143,12 @@ | |
| 143 | ** property is set to the tag/branch name found in the request. |
| 144 | ** |
| 145 | ** Only one of "tag" or "branch" modes will work at a time, and if |
| 146 | ** both are specified, which one takes precedence is unspecified. |
| 147 | */ |
| 148 | static signed char json_timeline_add_tag_branch_clause(Blob *pSql, |
| 149 | cson_object * pPayload){ |
| 150 | char const * zTag = NULL; |
| 151 | char const * zBranch = NULL; |
| 152 | char const * zMiOnly = NULL; |
| 153 | char const * zUnhide = NULL; |
| 154 | int tagid = 0; |
| @@ -220,11 +220,11 @@ | |
| 220 | ** applies one of them. |
| 221 | ** |
| 222 | ** Returns -1 if it adds a "before" clause, 1 if it adds |
| 223 | ** an "after" clause, and 0 if adds only an order-by clause. |
| 224 | */ |
| 225 | static signed char json_timeline_add_time_clause(Blob *pSql){ |
| 226 | char const * zAfter = NULL; |
| 227 | char const * zBefore = NULL; |
| 228 | int rc = 0; |
| 229 | zAfter = json_find_option_cstr("after",NULL,"a"); |
| 230 | zBefore = zAfter ? NULL : json_find_option_cstr("before",NULL,"b"); |
| 231 |