[HTTPS-Everywhere] Trailing whitespaces

[David Batley]

Trailing whitespace shouldn't cause any problems in xml and
javascript, but it is a bit untidy.

Running something like
   sed -i -r 's/\s+$//' $( find ~/code/https-everywhere/ -type f )
would make the appropriate changes. I can send a patch, but it's
probably easier for someone with repo access to just run the command.

--dave

On 23 February 2011 14:48, Christophe Jarry
<christophe.jarry at ouvaton.org> wrote:
> Dear developers,
>
> When I want to patch IceCat 3.6.13 source code to get the extracted version of
> https-everywhere, quilt gives me the following warnings:
>
>    Warning: trailing whitespace in lines 25,27,40,59,64,66,68,70 of
> chrome/content/rules/GoogleServices.xml
>    Warning: trailing whitespace in lines 10,22,61 of
> Changelog
>    Warning: trailing whitespace in lines 33,34,35,36,37,38,39,40 of
> install.rdf
>    Warning: trailing whitespace in line 376 of
> components/https-everywhere.js
>    Warning: trailing whitespace in line 37 of
> chrome/content/rules/GoogleAPIs.xml
>    Warning: trailing whitespace in lines
> 10,37,47,55,58,61,64,67,72,73,78,79,81,86,89,92,95,102,104,108,112,116 of
> chrome/content/rules/Google.xml
>    Warning: trailing whitespace in line 1 of
> chrome/content/rules/00README
>    Warning: trailing whitespace in lines 2,20 of
> chrome/content/rules/Twitter.xml
>    Warning: trailing whitespace in lines 23,24,26,27 of
> chrome/content/rules/Amazon-off-by-default.xml
>    Warning: trailing whitespace in lines 13,16,22,24,27,29 of
> chrome/content/rules/Facebook.xml
>    Warning: trailing whitespace in line 15 of
> chrome/content/rules/WordPress.xml
>    Warning: trailing whitespace in lines 12,13,32 of
> chrome/content/rules/GMX.xml
>    Warning: trailing whitespace in lines 17,22,35,36,38,39 of
> chrome/content/preferences.xul
>    Warning: trailing whitespace in line 10 of
> chrome/content/rules/Live.xml
>    Warning: trailing whitespace in lines 258,259,283,328,332,334,345,348 of
> chrome/content/code/HTTPSRules.js
>
> I think it would be good to fix this, thanks.
>
> Christophe
> _______________________________________________
> HTTPS-everywhere mailing list
> HTTPS-everywhere at mail1.eff.org
> https://mail1.eff.org/mailman/listinfo/https-everywhere
>