Re: com web/bugs: Trim trailing whitespaces: include/query.php templates/search-rdf.php www/admin/index.php www/bug.php www/report.php

On 14/12/2018 21:31, Kalle Sommer Nielsen wrote:

Den fre. 14. dec. 2018 kl. 21.12 skrev Lester Caine <lester@lsces.co.uk>:

Early strip processing removed the various options of 'new line' but replacing them with a single space was not necessarily the right answer, which is why many scripts have a manual space at the end of each line of text. Just explaining why those spaces had originally been the correct coding style. There has been a debate recently on extending from an 80 character line default, and stripping line feeds to pad out a longer line length may well need these spaces reinstating ... just a matter of preferred coding style when working with SQL rather than other scripts.

If you found a bug with the patch, then report it at bugs.php.net so we can look into it, if not then please keep the fuzz off the list, thank you

The comment should be read in the same way as with other comments on unnecessary changes to the code simply to address a new style of coding! But as with other style changes much of the original rational has never been documented at all? -- Lester Caine - G8HFL ----------------------------- Contact - https://lsces.co.uk/wiki/?page=contact L.S.Caine Electronic Services - https://lsces.co.uk EnquirySolve - https://enquirysolve.com/ Model Engineers Digital Workshop - https://medw.co.uk Rainbow Digital Media - https://rainbowdigitalmedia.co.uk