On Mon, Jul 18, 2011 at 10:48 AM, Jeffrey Yasskin <[email protected]> wrote: > On Mon, Jul 18, 2011 at 10:05 AM, Jeffrey Yasskin <[email protected]> wrote: >> On Mon, Jul 18, 2011 at 9:59 AM, Sean Hunt <[email protected]> >> wrote: >>> On 07/18/11 09:54, Jeffrey Yasskin wrote: >>>> I've attached the latest version of this patch. >>> >>> Perhaps it would be best to split the narrowing conversion and the C++0x >>> compat refactor into separate patches? >> >> You mean I should set the new warning to DefaultIgnore and revert the >> unittest fixes in this patch, and turn on the warning in a separate >> patch? > > Here's a version with the warning off by default.
Here's another version with all the previously-committed pieces excluded, and with the warning off by default. The latest version is also still at http://codereview.appspot.com/4698047/. Jeffrey
cxx0x_narrowing_error.diff
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
