fix handling of '#pragma pack(pop [, id])'

Zack Weinberg zackw@Stanford.EDU
Fri Sep 29 22:21:00 GMT 2000


On Sat, Sep 30, 2000 at 12:06:50AM -0500, Mike Coleman wrote:
> "Zack Weinberg" <zackw@stanford.edu> writes:
> > You must not use a variable to hold an error message, or message
> > translation will break.  You have to do it like this:
> 
> Ok, I didn't know about this.
>
> > Better you should set align to zero at the beginning of the function
> > and leave the switch alone.
> 
> That would also work.  I put the 'action != pop' check in to make it clearer
> to the reader that the switch didn't apply in the 'pop' case.  Personal
> taste--your way is fine with me.

Your way is fine with me, too.

> Do I need to submit a revised patch?

Please.  And maybe some test cases?

zw


More information about the Gcc-patches mailing list