Hello Andrew!

24 Nov 17 05:58, you wrote to Ward Dossche:

 AL> Besides, the coding and testing time for this change will be minimal, as
 AL> the code is basically already written.

 I found that there are 4 superfluous curly brackets in my code example.
 This is what the whole case should look like:

        case CFG_PUBLISH:
            NewExtWDay = xlate_switch(strupper(args[0]), DOWSwitchTab);
            if (NewExtWDay == -1)
            {
                mklog(LOG_ERROR, "%s -- Invalid day of week '%s'",
                        cfgline, args[0]);
                mode = -1;
                break;
            }
            time(&thetime);
            mytime = localtime(&thetime);
            if (NewExtWDay == 7)
                NewExtWDay = mytime->tm_wday;
            if (NewExtWDay == 8)
                NewExtWDay = mytime->tm_wday + 1;
            NewExtWDay = (NewExtWDay + 1) % 7; /* publishing day->new
                                                  filextension day */
            break;


For completeness the is the other change in it's complete context.

static const struct switchstruct DOWSwitchTab[] = {
    {"SUNDAY", 3, 0},
    {"MONDAY", 3, 1},
    {"TUESDAY", 3, 2},
    {"WEDNESDAY", 3, 3},
    {"THURSDAY", 3, 4},
    {"FRIDAY", 3, 5},
    {"SATURDAY", 3, 6},
    {"TODAY", 3, 7},
    {"TOMORROW", 3, 8},
    {NULL, 0, -1}
};

Kees

--- GoldED+/LNX 1.1.5
 * Origin: As for me, all I know is that, I know nothing. (2:280/5003.4)