Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
AW: [cdt-dev] Error Parser Manager removes leading whitespace

Hi Pete,
 
Pete MacLiesh wrote:
>
> It would actually be nice to eat continuations forwards 
> rather than getting them one at a time. We have a tool that 
> pumps out dissertations like
> 
> Warning (TIE_WARN_UNUSED) foo.tie:27
>     the dissertation starts
>     here and goes on for
>     several lines making the
>     previous line alone not useful
> 
> 
> So if an error parser could peek forwards and optionally eat 
> continuations that would make it more robust. Probably not 
> easy the way it is stuctured as a stream though.

sorry if I misunderstood your posting, but the above seems a bit of a
digression :). 

I don't have an issue with receiving a "line at a time" from the Error
Parser Manager. My issue with the Error Parser Manager is, that it trims
the leading spaces of each line of the tool output _before_ they are
passed to our own Error Parser.

The continuation detection problem can be solved nicely, without peeking
beyond the current line, by making the Error Parser stateful (which some
might say is the same as "peeking backwards" ;). This however is not
something that affects the Error Parser Manager in any way.

Just my 2 cents worth of opinion :).

Kind regards,
Elias.


Back to the top