Gabriel Dos Reis - Re: [PING^3] Wconversion: fixes for C++ front-end (original) (raw)
This is the mail archive of the gcc-patches@gcc.gnu.orgmailing list for the GCC project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
- From: Gabriel Dos Reis
- To: Manuel López-Ibáñez
- Cc: gcc-patches
- Date: 08 Feb 2007 05:58:54 -0600
- Subject: Re: [PING^3] Wconversion: fixes for C++ front-end
- References: <6c33472e0702060114o72301234qf3348f22dc18f36e@mail.gmail.com> <6c33472e0702061156i55a18d1ftbc5b2c0d40e5b9ce@mail.gmail.com> <Pine.GSO.4.58.0702061616260.2333@sun> <6c33472e0702061455u49753f4ofce9650e209e11f6@mail.gmail.com> <6c33472e0702061506h495674d8gf548652ebd2fa@mail.gmail.com> <Pine.GSO.4.58.0702061730530.2333@sun> <6c33472e0702061629h6d54b610gaa403f990b3491c8@mail.gmail.com> <87hcty4wwb.fsf@soliton.cs.tamu.edu> <6c33472e0702070113y12c94ad0g31bfa510f59f7117@mail.gmail.com> <Pine.GSO.4.58.0702070954130.17820@sun> <6c33472e0702080337l1f443a35o14d286ce7b134e0e@mail.gmail.com>
"Manuel López-Ibáñez" lopezibanez@gmail.com writes:
[...]
| Another problem is that I would need to encapsulate the warning logic | in convert_and_check on its own function, so it can be called from | convert_and_check and ocp_convert. That is fine except for the fact | that the C++ front-end already uses convert_and_check in a few places, | which in turn calls convert, which in turn calls ocp_convert, that | would generate warnings again. How could we fix this situation?
Make a list of all the places where the C++ front-end calls convert_and_check, document why. Make a list of all the places where the C++ front-end does not call convert_and_check (that is already contained in your patch). Document the differences in the "kinds" of conversions being checked in both cases. That should give you a good start.
-- Gaby
- Follow-Ups:
- Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Manuel López-Ibáñez
- Re: [PING^3] Wconversion: fixes for C++ front-end
- References:
- [PING^3] Wconversion: fixes for C++ front-end
* From: Manuel López-Ibáñez - Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Manuel López-Ibáñez - Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Gabriel Dos Reis - Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Manuel López-Ibáñez - Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Manuel López-Ibáñez - Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Gabriel Dos Reis - Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Manuel López-Ibáñez - Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Gabriel Dos Reis - Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Manuel López-Ibáñez - Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Gabriel Dos Reis - Re: [PING^3] Wconversion: fixes for C++ front-end
* From: Manuel López-Ibáñez
- [PING^3] Wconversion: fixes for C++ front-end
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |