Hi Ted, Ted Unangst wrote on Sun, Jan 13, 2019 at 08:23:01PM -0500: > Ingo Schwarze wrote:
>> Feel free to either commit the complete patch below or commit >> only your part and optionally OK one, two, or three of my additional >> changes. > These look ok with one comment. Committed, thanks for checking. >> @@ -141,8 +141,11 @@ mpset(char *mptoparse) >> mmsg = strchr(mpath, '?'); >> if (mmsg) { >> *mmsg = '\0'; >> - mmsg++; >> + if (*++mmsg == '\0') >> + mmsg = NULL; >> } > I would prefer the increment remain separate from the test. Sure, done. Yours, Ingo