Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] Anyone working on "Add Include" refactoring?

Thanks, Chris.  
On Fri, Jan 30, 2009 at 11:11 AM, Chris Recoskie <recoskie@xxxxxxxxxx> wrote:

We discussed some of the inherent problems with this feature at the Summit this past fall. If you haven't already, it would be worth reading the minutes (even though they're a bit short).

http://wiki.eclipse.org/CDT/summitfall2008/Indexing_Session#Add_Include_action

===========================
Chris Recoskie
Team Lead, IBM CDT and RDT
IBM Toronto


Inactive hide details for Tom Ball <tball@xxxxxxxxxx>Tom Ball <tball@xxxxxxxxxx>



To

"CDT General developers list." <cdt-dev@xxxxxxxxxxx>

cc


Subject

Re: [cdt-dev] Anyone working on "Add Include" refactoring?

Doh! I keep forgetting that JDT and CDT split their refactorings into source and refactoring menu lists, and so just assumed that because it wasn't in the refactoring menu or in one of the refactoring packages, it didn't exist. Thanks for the pointer -- although the existing Add Include has its problems it should be enough for the other refactoring to use. As I find blockers in Add Include, I'll submit separate patches.

Tom

On Fri, Jan 30, 2009 at 10:26 AM, Sergey Prigogin <eclipse.sprigogin@xxxxxxxxx> wrote:
    There is an existing Add Include command, but it's riddled with problems. See for example 236530 and 255952. I had some ideas regarding Add Include. We can meet to talk about it if you like.

    -sergey

    On Fri, Jan 30, 2009 at 9:47 AM, Tom Ball <tball@xxxxxxxxxx> wrote:
    I'm working on a C++ refactoring similar to the "Change Method Signature" one for Java, and belatedly realized that restricting the list of valid types to choose from to those already resolved by the AST makes the refactoring much less useful. Is anyone working on the C++ equivalent of Java's "Add Import" and/or "Organize Imports" refactorings? If not, should I switch over and work on them first?

    Tom

    _______________________________________________
    cdt-dev mailing list

    cdt-dev@xxxxxxxxxxx
    https://dev.eclipse.org/mailman/listinfo/cdt-dev



    _______________________________________________
    cdt-dev mailing list

    cdt-dev@xxxxxxxxxxx
    https://dev.eclipse.org/mailman/listinfo/cdt-dev
_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev


_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev


GIF image

GIF image


Back to the top