[cfe-dev] -fdiagnostics-format= design review : added enum to patch

Erik Olofsson Erik.Olofsson at hansoft.se
Thu May 19 00:39:02 PDT 2011


I think you intended this:
+    Opts.Format = 1;
+    Opts.ShowColumn = DiagnosticOptions::Msvc;

To be:
+    Opts.Format = DiagnosticOptions::Msvc;
+    Opts.ShowColumn = false;

At least in Visual Studio 2010 columns are supported with the following format:
t.c:(3,10)
I not entirely sure but I think you need ColNo-1 on MSVC

Also, shouldn't there be tests for this?

Regards,
Erik

From: cfe-dev-bounces at cs.uiuc.edu [mailto:cfe-dev-bounces at cs.uiuc.edu] On Behalf Of Andrew Fish
Sent: den 18 maj 2011 23:15
To: cfe-dev
Subject: Re: [cfe-dev] -fdiagnostics-format= design review : added enum to patch

Update patch to use an enum, per Doug's suggestion.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20110519/4b786d32/attachment.html>


More information about the cfe-dev mailing list