Re: [cdt-dev] Re: [cdt-patch] CModelBuilder adds line numbers + JUnit te

["Douglas Schaefer" <dschaefe@xxxxxxxxxx>]

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         cc:                 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
cdt-dev@xxxxxxxxxxx
http://dev.eclipse.org/mailman/listinfo/cdt-dev