On Mon, Mar 4, 2019 at 7:18 AM Duy Nguyen <pclouds@xxxxxxxxx> wrote: > You probably should mention about the extra _() too (e.g. "while at > there, mark more strings for translation") just to be clear it's > intentional. Good call. Will update. > > +#include "parse-options.h" > > +#include "string-list.h" > > Everything looks good, except that I can't find where this > string-list.h is needed. Maybe just try to remove it and build to be > sure. You're right, string-list.h was not used. Removed. Brandon