Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] comment line color

> 
> This worked like a charm.  Thank you very much for saving me the trouble
> of tracking down the change.  Tracking down fixes when there is no patch
> is Bugzilla nor on cdt-patch is a real pain :-(
> 

Yes, got no solution for that.  we've try to maintain a plugin/ChangeLog file
that indicates the PR and some quick rational on the fix.  It help folks
to quickly locate a PR then they have a better of where/when to look.
But it is not consistent across plugins.

> If you don't mind forwarding the related PRs, I would appreciate it.
> I'm not familiar with the issues you allude to.
> 

Allright, you can get more details in the PRs(see below).

C/C++ environment is much more dynamic then Java, i.e.
*.cpp is not set in stone vs Java language(defines only *.java).

So to overcome this, Sam(timesys) made the first draft for the C/C++ File types(resolverModel).
 (1) The problem for 2.0.1(CDT) is that the other parties were not reacting to it, so
     if you say the file *.pc is now a C++ source file,  the indexer/CoreModel etc ...
     did not react until you close/reopen the project.
 (2) Problem with integration with Eclipse.  Example: you make a diff of
     file "iostream" with another "iostream" version, you will not get
     C/C++ diff viewer, but the default text. Since eclipse has no
     idea that "iostream" is a C++ file.
 (3) other miner fixes to the UI/Core of ResolverModel(only in 2.1)

Meanwhile, Eclipse-3.0 is coming with a new scheme "content type" extension,
see IContentManager.  This would let plugins define a contentType extension
that can be use as an attribute to other extension like the Editor,
the diff viewer, team etc ...
Hopefully will be able to move to that scheme for better integration, but
it is only schedule for 3.1.  The implementation in 3.0 is incomplete.
So probably CDT-2.1 will not be able to provide a good solution.


Relevant PRs:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=72796
https://bugs.eclipse.org/bugs/show_bug.cgi?id=69640
https://bugs.eclipse.org/bugs/show_bug.cgi?id=72821
https://bugs.eclipse.org/bugs/show_bug.cgi?id=72396
https://bugs.eclipse.org/bugs/show_bug.cgi?id=72395




Back to the top