Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] Re: [cdt-patch] CModelBuilder adds line numbers + JUnit test

So what you are saying is leave the line information in CElement and just change the interfaces around.  Or am I mistaken?

Word from Hoda is that there is still some code lying around expecting the line information to be sitting in CElement.  Changing the implementation of CElement might be too major of a change for this stage of 1.1, unless we have broken something else in 1.1.  Hoda can comment more.

Doug Schaefer
Rational Software - IBM Software Group
Ottawa, Ontario, Canada

"Alain Magloire" <alain@xxxxxxx>
Sent by: cdt-dev-admin@xxxxxxxxxxx

04/21/2003 01:58 PM
Please respond to cdt-dev

        To:        cdt-dev@xxxxxxxxxxx
        Subject:        Re: [cdt-dev] Re: [cdt-patch] CModelBuilder adds line numbers + JUnit test

> This is a multipart message in MIME format.
> --=_alternative 005F2CE285256D0F_=
> Content-Type: text/plain; charset="US-ASCII"
> I'll agree and take it a step furthur.  I don't think we've given this
> line number stuff enough thought and it feels like we are rushing to quick
> answers.
> However, in order to get the release completed, is what we have broken, or
> can we release it and work on a better solution in May?

I think, we have an already good framework, it corresponds to the one of the JDT
line information is provided through the ISourceRange interface.  Classes
that need to provide line information, implements the ISourceReference interface.

So the problems:
- the new Parser was not providing line information.
 * But this is corrected(or will be by John C. when possible).
- Removing ICElement.setStartLine(), the correct way to retrieve offset are line
 information should be trough the ISourceRange interface.
 * this will be corrected(will be corrected by Hoda).

I am not aware of any other difficulties.

cdt-dev mailing list

Back to the top