[PATCH v3 1/6] xmllite: Use empty_element to store various names.

Nikolay Sivov nsivov at codeweavers.com
Sun Nov 3 13:50:52 CST 2019


Hi, Jeff.

Sorry for the delay.

I think this looks good in principle, my suggestion would be to do a 
deeper cleanup first. Especially for 'empty_element' field.

Right now it's used for things that are not elements, with name 
suggesting otherwise. Using 'struct element' for it is also misleading 
for the same reason.

Maybe we can keep is as is, and use strictly for empty elements, because 
it's used together with element stack. Then we could add another 
structure, without defined type for example as a part of reader struct, 
to hold temporary data for "current" node - strings and position.




More information about the wine-devel mailing list