Review Request: Add Switch to buddy plugin which always swichting between cpp and header file [1/2]

Milian Wolff mail at milianw.de
Thu Nov 15 20:29:34 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107310/#review22048
-----------------------------------------------------------



interfaces/ibuddydocumentfinder.h
<http://git.reviewboard.kde.org/r/107310/#comment16969>

    Please use a QVector<KUrl> here, see also:
    
    http://marcmutz.wordpress.com/effective-qt/containers/



plugins/switchtobuddy/switchtobuddyplugin.cpp
<http://git.reviewboard.kde.org/r/107310/#comment16968>

    I'd rewrite this to reduce the amount of indentation required:
    
    EditorContext* ctx = ...;
    if (!ctx) {
      return ContextMenuExtension();
    }
    
    Furthermore, please follow the coding style guidelines for new code, esp. put conditional and loop braces on the same line. Also add the braces everywhere, also for single-line loops and conditionals.
    
    see http://techbase.kde.org/Projects/KDevelop4/CodingStyle



shell/tests/shellbuddytest.cpp
<http://git.reviewboard.kde.org/r/107310/#comment16970>

    If you don't have tests using this, you can leave it empty alltogether, no?


- Milian Wolff


On Nov. 14, 2012, 8:08 a.m., André Stein wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107310/
> -----------------------------------------------------------
> 
> (Updated Nov. 14, 2012, 8:08 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> This request adds a new plugin "Switch to buddy" which enables in the c++ language plugin a new context menu action to switch to corresponding header or implementation file. This request is split in two parts because it needs changes in kdevelop too (see Request 107311).
> 
> Changes made:
> - A new function has been added to IBuddyDocumentFinder to allow language plugins to implement logic to find buddy or related files.
> - The new plugin "switch to buddy" implements a context menu extension which is only trigged if at least one buddy can be found. To get possible buddies IBuddyDocumentFinder is used using the editor's current document's mime type.
> - See plugin
> 
> This implements a feature known from many IDEs. Using the Buddy interface and a separate plugin I think it's rather clean and generic.
> 
> 
> Diffs
> -----
> 
>   interfaces/ibuddydocumentfinder.h 7807df7 
>   plugins/CMakeLists.txt 8aac3ae 
>   plugins/switchtobuddy/CMakeLists.txt PRE-CREATION 
>   plugins/switchtobuddy/Messages.sh PRE-CREATION 
>   plugins/switchtobuddy/kdevswitchtobuddy.desktop PRE-CREATION 
>   plugins/switchtobuddy/switchtobuddyplugin.h PRE-CREATION 
>   plugins/switchtobuddy/switchtobuddyplugin.cpp PRE-CREATION 
>   shell/tests/shellbuddytest.cpp 981daa3 
> 
> Diff: http://git.reviewboard.kde.org/r/107310/diff/
> 
> 
> Testing
> -------
> 
> Manual testing.
> 
> 
> Screenshots
> -----------
> 
> Context menu in action
>   http://git.reviewboard.kde.org/r/107310/s/830/
> 
> 
> Thanks,
> 
> André Stein
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20121115/43fdac9b/attachment.html>


More information about the KDevelop-devel mailing list