[PATCH 2/2] regedit: Handle comment lines with leading spaces and tabs

Hugh McMaster hugh.mcmaster at outlook.com
Fri Mar 31 07:31:37 CDT 2017


On Friday, 31 March 2017 11:07 PM, Alexandre Julliard wrote:
> You can also ignore white space before comments, but still return it for
> other lines, if there is a case where the callers would want that.
> Otherwise getting rid of whitespace right away is fine. A few test cases
> with leading whitespace may be a good idea.

Somewhat strangely, we actually call isspaceW() in processSetValue() before we
get the registry value name. But it is already too late. We need to check for
whitespace in processRegEntry() or earlier; otherwise, the import fails.

So I think handling whitespace in get_lineA/W() will be far more efficient
and easier to maintain. But to be certain, I'll add some tests with whitespace
variations first.

Thanks for the advice.

--
Hugh McMaster


More information about the wine-devel mailing list