On Sat, Sep 21, 2024 at 08:08:29PM -0600, Sandra Loosemore wrote:
> On 9/20/24 01:41, Jakub Jelinek wrote:
> > > +
> > > + /* Check for unknown properties. */
> > > if (omp_ts_map[ts_code].valid_properties == NULL)
> > > continue;
> > > -
> >
> > Why?
>
> Why what? I made this change because when I added another check in this
> loop I was temporarily confused about the control flow and I thought this
> would help. If you're asking why it's doing the null check here, it's
> because it doesn't make sense to check properties on all selectors, like
> properties that can have implementation-defined values for other compilers.
I meant why the empty line was removed?
Jakub
