[TYPO3-core] RFC #13729: Cleanup: t3lib_div CGL Cleanup: Missing spaces around string concatenator
Sebastian Gebhard
sebastiangebhard at hoch2.de
Fri Mar 5 10:31:42 CET 2010
At every "array()" you added a space before the Brackets.
Despite that you convinced me. I'd give +1 for commiting your patch
Am 03.03.10 18:27, schrieb Steffen Kamper:
> Hi,
>
> Sebastian Gebhard schrieb:
>> Am 03.03.10 16:37, schrieb Steffen Kamper:
>>> don't you have a code formatter? Than it can be done with one click. I
>>> configured Zend to get core CGL, only missing 2-3 issues.
>>> I would prefer to have RFC per file.
>>>
>>> I don't want to block you, but manually doing this is time consuming and
>>> stupid work, so let's find the tools doing this automatic.
>>
>> I used search/replace with regular expressions and some handwork to
>> insert the spaces. If you know a tool for Eclipse for CGLifying, i'd
>> be very thankful.
>>
>> To limit this RFC these spaces was not only because of how much work
>> it is to completely CGLify the file. Completely CGLifying would mean
>> almost every line would be changed, which would make the review
>> process quite hard.
>
> just as example - i used the zend code formater on t3lib_div. it's safe
> that no functionality has changed, and it was work of 10 seconds.
>
> Sure there are 2-3 places where custom linebreaks were done before, but
> all the spaces / tabs etc were converted to CGL.
>
> The code formater in Zend is unique, there is no one for eclipse (they
> want you to buy Zend), result is much much better than before.
>
> vg Steffen
More information about the TYPO3-team-core
mailing list