Actually, it saves only one compared to the original code. It also
save the declaration and possible assignment to a local variable, for
what it's worth.
EdB
On Thu, Sep 12, 2013 at 12:30 PM, Darrell Loverin <dlove...@adobe.com> wrote:
> The local variable saves two lookups in rowIndicators. That would be
> enough for me.
>
>
> -Darrell
>
>
> On 9/12/13 4:34 AM, "Erik de Bruin" <e...@ixsoftware.nl> wrote:
>
>>I'd go with:
>>
>>var rowIndicators:Object = cellSelectionIndicators[rowData.uid];
>>if (rowIndicators && rowIndicators[columnIndex])
>>{
>> o = rowIndicators[columnIndex];
>> if (o.parent)
>> o.parent.removeChild(o);
>>
>> delete rowIndicators[columnIndex];
>> if (!atLeastOneProperty(rowIndicators))
>> delete cellSelectionIndicators[rowData.uid];
>>}
>>
>>No need for the extra local variable, it seems.
>>
>>EdB
>>
>>
>>
>>On Thu, Sep 12, 2013 at 10:27 AM, Justin Mclean
>><jus...@classsoftware.com> wrote:
>>> Hi,
>>>
>>> Looks much better.
>>>
>>> Justin
>>
>>
>>
>>--
>>Ix Multimedia Software
>>
>>Jan Luykenstraat 27
>>3521 VB Utrecht
>>
>>T. 06-51952295
>>I. www.ixsoftware.nl
>
--
Ix Multimedia Software
Jan Luykenstraat 27
3521 VB Utrecht
T. 06-51952295
I. www.ixsoftware.nl
