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

Reply via email to