[TYPO3-core] RFC#15562: Bug/Nobrainer: PHP warning str_pad() in t3lib_div
François Suter
fsu-lists at cobweb.ch
Sun Sep 5 21:40:56 CEST 2010
Hi,
> I never intended to touch any other things than these quotes.
> But I had a response
> (http://lists.typo3.org/pipermail/typo3-team-core/2010-August/042554.html)
> that requested me to fix some CGL issues.
> I don't think it makes sense to fix CGL only for 2 lines, so I adjusted the
> whole "paragraph".
> But if Core Team says to keep the other lines, it's ok for me of course. np
IMO it's ok to fix other lines, but the risk is that your changes become
hard to make out in the midst of purely CGL-related fixes. One thing you
can do in such cases is to submit 2 versions of your patch, one normal
and one without white-space differences (svn diff -x -w).
Cheers
--
Francois Suter
Cobweb Development Sarl - http://www.cobweb.ch
More information about the TYPO3-team-core
mailing list