[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [pdf-devel] [patch] convert LZW filter to new API
From: |
Brad Hards |
Subject: |
Re: [pdf-devel] [patch] convert LZW filter to new API |
Date: |
Thu, 25 Jun 2009 18:00:59 +1000 |
User-agent: |
KMail/1.9.10 |
On Thursday 25 June 2009 04:32:17 am address@hidden wrote:
> -AM_CFLAGS = -Wall -fno-strict-aliasing $(LIBGCRYPT_CFLAGS)
> +AM_CFLAGS = -Wall -fno-strict-aliasing -Wconversion $(LIBGCRYPT_CFLAGS)
>
> The -Wconversion option makes gcc to generate a lot of warnings in
> both header files included from gnulib and our source code. Is there
> a particular reason to add that switch?
That probably shouldn't have been merged. It often shows some useful bugs, but
also shows some unwarranted warnings.
In the longer term, we should be using some more targeted warning flags, and
trying to fix the warnings.
Brad
- [pdf-devel] [patch] convert LZW filter to new API, Brad Hards, 2009/06/24
- Re: [pdf-devel] [patch] convert LZW filter to new API, Juan Pedro Bolivar Puente, 2009/06/24
- Re: [pdf-devel] [patch] convert LZW filter to new API, jemarch, 2009/06/24
- Re: [pdf-devel] [patch] convert LZW filter to new API, Juan Pedro Bolivar Puente, 2009/06/24
- Re: [pdf-devel] [patch] convert LZW filter to new API, Brad Hards, 2009/06/25
- Re: [pdf-devel] [patch] convert LZW filter to new API, Juan Pedro Bolivar Puente, 2009/06/25
- Re: [pdf-devel] [patch] convert LZW filter to new API, jemarch, 2009/06/25
- Re: [pdf-devel] [patch] convert LZW filter to new API, Juan Pedro Bolivar Puente, 2009/06/25
- Re: [pdf-devel] [patch] convert LZW filter to new API, jemarch, 2009/06/26
Re: [pdf-devel] [patch] convert LZW filter to new API,
Brad Hards <=