[TYPO3-core] RFC #13729: Cleanup: t3lib_div CGL Cleanup: Missing spaces around string concatenator

Steffen Kamper info at sk-typo3.de
Wed Mar 3 18:27:28 CET 2010


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
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: t3lib_div_cgl.diff
URL: <http://lists.typo3.org/pipermail/typo3-team-core/attachments/20100303/3d889c49/attachment.asc>


More information about the TYPO3-team-core mailing list