[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