On Tue, Jan 14, 2014 at 06:52:00PM +0100, Florian Weimer wrote:
> On 01/13/2014 09:48 PM, Marek Polacek wrote:
> >+bool
> >+pointer_to_zero_sized_aggr_p (tree t)
> >+{
> >+ t = strip_pointer_operator (t);
> >+ if (RECORD_OR_UNION_TYPE_P (t)
> >+ && TYPE_SIZE (t)
> >+ && integer_zerop (TYPE_SIZE (t)))
> >+ return true;
> >+ return false;
> >+}
>
> I think you can just return the value of the condition, there's no
> need for the if statement.
Oops, yeah, forgot to change that. Will do it in next patch.
Marek