[TYPO3-core] RFC Feature #16189: Optimize stdWrap usage for TypoScript content element CLEARGIF
Marcus Schwemer
typo3 at schwemer.de
Thu Nov 4 00:42:11 CET 2010
Hi,
+1 by reading and testing.
Regards,
Marcus
Susanne Moog schrieb am 03.11.10 19:18:
> On 01.11.2010 08:02, JoH asenau wrote:
>>> i suggest this layout for better readability:
>>>
>>> $var .... ==== TRUE
>>> ? the longer truee part
>>> : the longer else part;
>>>
>>> From performance this operator isn't faster than if/else, it's only a
>>> shortcut and is processed the same.
>>>
>>> So for longer expressions if/else indeed are better readable.
>> And for expressions that keep the value for on of the cases as well.
>>
>> So attached is v3 of the patch, that takes care of these.
>> If this is OK, I will reformat the other patches as well and try to improve readability and to get rid of superfluous stuff.
>
> I don't like
>
> + if(!$width)
> + $width = 1;
> +
>
> without brackets, so attached one with brackets.
>
> Other than that: +1 by reading and testing,
>
> Susanne
>
>
More information about the TYPO3-team-core
mailing list