[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Tinycc-devel] Preprocessor directive "pragma"
From: |
Suote127 |
Subject: |
Re: [Tinycc-devel] Preprocessor directive "pragma" |
Date: |
Fri, 20 Aug 2021 22:11:18 +0800 |
User-agent: |
Mutt/1.9.4 (2018-02-28) |
On Fri, Aug 20, 2021 at 01:51:26PM +0200, grischka wrote:
> That's good, but as you may guess, none of the tests that so far
> exist was made specifically to test your new feature. See for
> example:
>
> #pragma pack(push+1)
> #pragma pack(pop)
> #pragma pack(push)
> struct foo { char c; int d; };
> #pragma pack(pop)
> main() { printf("size: %d\n", sizeof (struct foo)); }
>
> should notify the user about the typo, and when you replace
> the '+' by a ',' it should print 'size: 8' (not '5').
Yes.This is my mistake.I made a new patch and it fixes this problem.
>
> You can add one or two tests in 60_errors_and_warnings.c,
> if you want.
I did so.Now error message for above example is similar to the other pragma
options.And a test was added.
Please ONLY apply the patch I sent in this mail.
Suote127
pragma_pack.patch
Description: Text Data