flex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Cyrill Zadra <cyrill.za...@gmail.com>
Subject Re: [?Falcon?] AdvandedDataGrid error
Date Thu, 12 Sep 2013 10:13:17 GMT
Oh right.. cool! And  ./mini_run.sh tests/components/AdvancedDataGrid/
passes with 0 fails :). Thanks.


On Thu, Sep 12, 2013 at 11:54 AM, Erik de Bruin <erik@ixsoftware.nl> wrote:

> That null check is already in the first if-statement: &&
> rowIndicators[columnIndex]. If 'rowIndicators[columnIndex]' is null it
> will never enter the block with the assignment to 'o', therefor 'o'
> can't be null.
>
> EdB
>
>
>
> On Thu, Sep 12, 2013 at 11:41 AM, Cyrill Zadra <cyrill.zadra@gmail.com>
> wrote:
> > Shouldn't there be an additional null check?
> >
> >   o = rowIndicators[columnIndex];
> >   if (o != null && o.parent)
> >     o.parent.removeChild(o);
> >
> >
> > On Thu, Sep 12, 2013 at 10:34 AM, Erik de Bruin <erik@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
> >> <justin@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
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message