[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
|
Re: [cdt-patch] Patch for PR 44892
|
Yellow,
Sorry for the delay.
> This patch (again applied against CDT 1.2, and I noticed that you
> didn't apply the original patch against CDT 1.2 so there is a bit
> of cross talk, let me know if this is a problem) provides a bit of
> clean-up to the C/C++ Editor context menu to make it match the
> look/feel of the Java Editor context menu. It also adds in a new
> command/action which will provide the Show In CView capability from
> the context menu.
>
> Hits CEditor, bunch of properties and creates a new OpenInCViewAction.
>
> For the ChangeLog:
> Add missing support for the C Editor context menu to show the entry
> in the C/C++ Project view. Reorganized the context menu to include
> source related sub-menus.
Will it be possible to have 2 different patches, one for the head
and the other for Target MileStone 1.2.1?
The problem with that patch:
I'm not sure you want to put menus like
"refactoring"/"Format"/"Reorganise include". Do not think
that they will be ever part of any 1.2.x, so why bother
to put code for that ?
And even for the head, there is actually no code dealing with
these issues, basically noops.
Merci bucket.
