[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] clean up release notes

From: Stefan <luke1410_at_posteo.de>
Date: Mon, 16 May 2016 14:29:34 +0200

On 5/16/2016 14:15, Branko Čibej wrote:
> On 16.05.2016 14:09, Stefan wrote:
>> On 5/16/2016 13:43, Branko Čibej wrote:
>>> On 16.05.2016 13:19, Stefan wrote:
>>>> Hi,
>>>>
>>>> following patch fixes some more W3C validation issues inside the release
>>>> notes
>>>>
>>>> [[[
>>>> Make the release notes more compliant with the XHTML 1.1 standard.
>>>>
>>>> * docs/release-notes/1.6.html
>>>> (Testsuite fails): add missing closing p-tag
>>>>
>>>> * docs/release-notes/1.7.html
>>>> (svn diff can show git-style diff annotations): close p-tag before ul
>>>>
>>>> * docs/release-notes/1.8.html
>>>> (Skelta style updates are now the default): drop invalid p-section
>>> What exactly did you do with the apostrophe in this file? It shows up as
>>> an invalid character for me ... note that the files have to be encoded
>>> in UTF-8, if your editor changes them to Latin-1, ditch your editor. :)
>>>
>>> See the first hunk in the diff (which, by the way, is not mentioned here
>>> in the log message.
>> Ouch. That was certainly unintended. Was caused by an incorrect setting
>> in WinMerge on my side (corrected and should no longer occur).
>>
>> Also I didn't intend to change the absence or presence of the BOM
>> (didn't realize my editor is set to drop that for html files).
>>
>> Both issues fixed with the attached patch.
> Looks good, please commit.

Committed in r1744038.

Regards,
Stefan

Received on 2016-05-16 14:29:43 CEST

This is an archived mail posted to the Subversion Dev mailing list.