[PATCH v3 3/3] msxml3: When saving XML use encoding specified in the processing instruction. (Resend)

Nikolay Sivov nsivov at codeweavers.com
Mon May 31 06:27:35 CDT 2021



On 5/31/21 1:10 PM, Dmitry Timoshkov wrote:
> Dmitry Timoshkov <dmitry at baikal.ru> wrote:
>
>> Nikolay Sivov <nsivov at codeweavers.com> wrote:
>>
>>> Is save() part testable? Something close to what your application needs
>>> seems useful to have in tests. Does it load with one encoding, then
>>> changes it and saves?
>> The application creates XML in run-time, saves it and then loads from the file.
>> There's a test for this already in test_createProcessingInstruction(), however
>> testing encoding of the contents requires using some cyrillic instead of "test",
>> so I decided to avoid that.
> Do you need any other clarifications, or the patches might be accepted as is?
>
Let's have them as is. Would still be nice to have that saving test that
shows that correct encoding is used, as a follow up patch.



More information about the wine-devel mailing list