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