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

Reply via email to