This is the mail archive of the mailing 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]

Re: [PATCH] Caret diagnostics

On Tue, Apr 10, 2012 at 11:46 AM, Manuel LÃpez-IbÃÃez
<> wrote:
> On 10 April 2012 00:28, Jason Merrill <> wrote:
>> On 04/09/2012 04:01 PM, Manuel LÃpez-IbÃÅez wrote:
>>> * It uses Âthe default cutoff as max_width, whatever it is (as
>>> controlled by -fmessage-length).
>>> * It uses the pretty-printer. The text cannot (should not) wrap
>>> because we still print only max_width chars at most.
>> Hmm, I think if pp_line_cutoff is 0 and we're on a terminal, we still want
>> to use COLUMNS to limit how much of the source we print.
> Like this?

There is a novelty in this new version that I don't think we discussed
before: automatic expansion of tabs to 8 hard space characters.  That
number should not be hardcoded as there is no reason to believe a tab
character always expands to 8 space characters.  You should check
the environment first; if not present the default expansion number should
be a symbolic constant as opposed to a magic number sprinkled all over the
places.  I would also encourage you to introduce more abstraction to
reduce the size of diagnostic_show_locus.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]