Fossil SCM
Make azType in statrep.c a const table (which it actually is ...)
Commit
b43681dc227deda2bb4217394065ee2ef62afc28
Parent
53be87d7d712bff…
2 files changed
+1
-1
+3
-3
+1
-1
| --- src/statrep.c | ||
| +++ src/statrep.c | ||
| @@ -684,11 +684,11 @@ | ||
| 684 | 684 | { "By User", "byuser", RPT_BYUSER }, |
| 685 | 685 | { "By Week", "byweek", RPT_BYWEEK }, |
| 686 | 686 | { "By Weekday", "byweekday", RPT_BYWEEKDAY }, |
| 687 | 687 | { "By Year", "byyear", RPT_BYYEAR }, |
| 688 | 688 | }; |
| 689 | - const char *azType[] = { | |
| 689 | + const char *const azType[] = { | |
| 690 | 690 | "a", "All Changes", |
| 691 | 691 | "ci", "Check-ins", |
| 692 | 692 | "g", "Tags", |
| 693 | 693 | "e", "Tech Notes", |
| 694 | 694 | "t", "Tickets", |
| 695 | 695 |
| --- src/statrep.c | |
| +++ src/statrep.c | |
| @@ -684,11 +684,11 @@ | |
| 684 | { "By User", "byuser", RPT_BYUSER }, |
| 685 | { "By Week", "byweek", RPT_BYWEEK }, |
| 686 | { "By Weekday", "byweekday", RPT_BYWEEKDAY }, |
| 687 | { "By Year", "byyear", RPT_BYYEAR }, |
| 688 | }; |
| 689 | const char *azType[] = { |
| 690 | "a", "All Changes", |
| 691 | "ci", "Check-ins", |
| 692 | "g", "Tags", |
| 693 | "e", "Tech Notes", |
| 694 | "t", "Tickets", |
| 695 |
| --- src/statrep.c | |
| +++ src/statrep.c | |
| @@ -684,11 +684,11 @@ | |
| 684 | { "By User", "byuser", RPT_BYUSER }, |
| 685 | { "By Week", "byweek", RPT_BYWEEK }, |
| 686 | { "By Weekday", "byweekday", RPT_BYWEEKDAY }, |
| 687 | { "By Year", "byyear", RPT_BYYEAR }, |
| 688 | }; |
| 689 | const char *const azType[] = { |
| 690 | "a", "All Changes", |
| 691 | "ci", "Check-ins", |
| 692 | "g", "Tags", |
| 693 | "e", "Tech Notes", |
| 694 | "t", "Tickets", |
| 695 |
+3
-3
| --- src/style.c | ||
| +++ src/style.c | ||
| @@ -48,11 +48,11 @@ | ||
| 48 | 48 | const char *zName; /* Form query parameter */ |
| 49 | 49 | const char *zLabel; /* Label. Might be NULL for FF_MULTI */ |
| 50 | 50 | unsigned char eType; /* FF_ENTRY, FF_MULTI, FF_BINARY */ |
| 51 | 51 | unsigned char isDisabled; /* True if this control is grayed out */ |
| 52 | 52 | short int iSize; /* Width for FF_ENTRY. Count for FF_MULTI */ |
| 53 | - const char **azChoice; /* value/display pairs for FF_MULTI */ | |
| 53 | + const char *const *azChoice;/* value/display pairs for FF_MULTI */ | |
| 54 | 54 | const char *zFalse; /* FF_BINARY label when false */ |
| 55 | 55 | } aSubmenuCtrl[20]; |
| 56 | 56 | static int nSubmenuCtrl = 0; |
| 57 | 57 | #define FF_ENTRY 1 |
| 58 | 58 | #define FF_MULTI 2 |
| @@ -274,11 +274,11 @@ | ||
| 274 | 274 | nSubmenuCtrl++; |
| 275 | 275 | } |
| 276 | 276 | void style_submenu_multichoice( |
| 277 | 277 | const char *zName, /* Query parameter name */ |
| 278 | 278 | int nChoice, /* Number of options */ |
| 279 | - const char **azChoice, /* value/display pairs. 2*nChoice entries */ | |
| 279 | + const char *const *azChoice,/* value/display pairs. 2*nChoice entries */ | |
| 280 | 280 | int isDisabled /* True if this control is disabled */ |
| 281 | 281 | ){ |
| 282 | 282 | assert( nSubmenuCtrl < ArraySize(aSubmenuCtrl) ); |
| 283 | 283 | aSubmenuCtrl[nSubmenuCtrl].zName = zName; |
| 284 | 284 | aSubmenuCtrl[nSubmenuCtrl].iSize = nChoice; |
| @@ -313,11 +313,11 @@ | ||
| 313 | 313 | db_finalize(&q); |
| 314 | 314 | if( n>0 ){ |
| 315 | 315 | aSubmenuCtrl[nSubmenuCtrl].zName = zName; |
| 316 | 316 | aSubmenuCtrl[nSubmenuCtrl].zLabel = zLabel; |
| 317 | 317 | aSubmenuCtrl[nSubmenuCtrl].iSize = n/2; |
| 318 | - aSubmenuCtrl[nSubmenuCtrl].azChoice = (const char**)az; | |
| 318 | + aSubmenuCtrl[nSubmenuCtrl].azChoice = (const char *const *)az; | |
| 319 | 319 | aSubmenuCtrl[nSubmenuCtrl].isDisabled = 0; |
| 320 | 320 | aSubmenuCtrl[nSubmenuCtrl].eType = FF_MULTI; |
| 321 | 321 | nSubmenuCtrl++; |
| 322 | 322 | } |
| 323 | 323 | } |
| 324 | 324 |
| --- src/style.c | |
| +++ src/style.c | |
| @@ -48,11 +48,11 @@ | |
| 48 | const char *zName; /* Form query parameter */ |
| 49 | const char *zLabel; /* Label. Might be NULL for FF_MULTI */ |
| 50 | unsigned char eType; /* FF_ENTRY, FF_MULTI, FF_BINARY */ |
| 51 | unsigned char isDisabled; /* True if this control is grayed out */ |
| 52 | short int iSize; /* Width for FF_ENTRY. Count for FF_MULTI */ |
| 53 | const char **azChoice; /* value/display pairs for FF_MULTI */ |
| 54 | const char *zFalse; /* FF_BINARY label when false */ |
| 55 | } aSubmenuCtrl[20]; |
| 56 | static int nSubmenuCtrl = 0; |
| 57 | #define FF_ENTRY 1 |
| 58 | #define FF_MULTI 2 |
| @@ -274,11 +274,11 @@ | |
| 274 | nSubmenuCtrl++; |
| 275 | } |
| 276 | void style_submenu_multichoice( |
| 277 | const char *zName, /* Query parameter name */ |
| 278 | int nChoice, /* Number of options */ |
| 279 | const char **azChoice, /* value/display pairs. 2*nChoice entries */ |
| 280 | int isDisabled /* True if this control is disabled */ |
| 281 | ){ |
| 282 | assert( nSubmenuCtrl < ArraySize(aSubmenuCtrl) ); |
| 283 | aSubmenuCtrl[nSubmenuCtrl].zName = zName; |
| 284 | aSubmenuCtrl[nSubmenuCtrl].iSize = nChoice; |
| @@ -313,11 +313,11 @@ | |
| 313 | db_finalize(&q); |
| 314 | if( n>0 ){ |
| 315 | aSubmenuCtrl[nSubmenuCtrl].zName = zName; |
| 316 | aSubmenuCtrl[nSubmenuCtrl].zLabel = zLabel; |
| 317 | aSubmenuCtrl[nSubmenuCtrl].iSize = n/2; |
| 318 | aSubmenuCtrl[nSubmenuCtrl].azChoice = (const char**)az; |
| 319 | aSubmenuCtrl[nSubmenuCtrl].isDisabled = 0; |
| 320 | aSubmenuCtrl[nSubmenuCtrl].eType = FF_MULTI; |
| 321 | nSubmenuCtrl++; |
| 322 | } |
| 323 | } |
| 324 |
| --- src/style.c | |
| +++ src/style.c | |
| @@ -48,11 +48,11 @@ | |
| 48 | const char *zName; /* Form query parameter */ |
| 49 | const char *zLabel; /* Label. Might be NULL for FF_MULTI */ |
| 50 | unsigned char eType; /* FF_ENTRY, FF_MULTI, FF_BINARY */ |
| 51 | unsigned char isDisabled; /* True if this control is grayed out */ |
| 52 | short int iSize; /* Width for FF_ENTRY. Count for FF_MULTI */ |
| 53 | const char *const *azChoice;/* value/display pairs for FF_MULTI */ |
| 54 | const char *zFalse; /* FF_BINARY label when false */ |
| 55 | } aSubmenuCtrl[20]; |
| 56 | static int nSubmenuCtrl = 0; |
| 57 | #define FF_ENTRY 1 |
| 58 | #define FF_MULTI 2 |
| @@ -274,11 +274,11 @@ | |
| 274 | nSubmenuCtrl++; |
| 275 | } |
| 276 | void style_submenu_multichoice( |
| 277 | const char *zName, /* Query parameter name */ |
| 278 | int nChoice, /* Number of options */ |
| 279 | const char *const *azChoice,/* value/display pairs. 2*nChoice entries */ |
| 280 | int isDisabled /* True if this control is disabled */ |
| 281 | ){ |
| 282 | assert( nSubmenuCtrl < ArraySize(aSubmenuCtrl) ); |
| 283 | aSubmenuCtrl[nSubmenuCtrl].zName = zName; |
| 284 | aSubmenuCtrl[nSubmenuCtrl].iSize = nChoice; |
| @@ -313,11 +313,11 @@ | |
| 313 | db_finalize(&q); |
| 314 | if( n>0 ){ |
| 315 | aSubmenuCtrl[nSubmenuCtrl].zName = zName; |
| 316 | aSubmenuCtrl[nSubmenuCtrl].zLabel = zLabel; |
| 317 | aSubmenuCtrl[nSubmenuCtrl].iSize = n/2; |
| 318 | aSubmenuCtrl[nSubmenuCtrl].azChoice = (const char *const *)az; |
| 319 | aSubmenuCtrl[nSubmenuCtrl].isDisabled = 0; |
| 320 | aSubmenuCtrl[nSubmenuCtrl].eType = FF_MULTI; |
| 321 | nSubmenuCtrl++; |
| 322 | } |
| 323 | } |
| 324 |