<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/100874/">http://git.reviewboard.kde.org/r/100874/</a>
     </td>
    </tr>
   </table>
   <br />


<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for KDevelop.</div>
<div>By Martin Heide.</div>





<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">(This review request is based on https://git.reviewboard.kde.org/r/100795/ , to separate cleanup and buddy document features)
Works together with the review request https://git.reviewboard.kde.org/r/100796/ in kdevelop. Apply them both before compiling.

Add document buddy architecture :
Allow kdevelop language plugins to manage related documents (like foo.h and foo.cpp for cpp plugin)
Language plugins may implement IBuddyDocumentFinder to define the rules (how to know if two documents are buddies).
Language plugins that implement IBuddyDocumentFinder  must register themselves.
An option was added to the UIConfig to open documents side by side with its buddies.

Buddy documents work like this: Assume that the option is enabled in the config, and there exist the following tabs:
| foo.h | bar.cpp |
If now the user opens foo.cpp, then this tab will be placed to the right of foo.h.

Another feature: Assume that both buddy documents and the option "Open new tab after current" are enabled. There are the following tabs:
| *foo.h* | foo.cpp | bar.h |  (foo.h is the active tab)
Normally, if the user opens somefile.cpp now, the tab would be placed between foo.h and foo.cpp, which would separate the buddies. So we changed the bahavior: the new tab will be placed after foo.cpp instead.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Created shell-buddytest and sublime-documentcontrollertest. Both pass.
Fixed an error in sublime-areaoperationtest (which was not related to the buddy document feature, but to the option "Open new tab after current").</pre>
  </td>
 </tr>
</table>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>interfaces/CMakeLists.txt <span style="color: grey">(170cf63)</span></li>

 <li>interfaces/ibuddydocumentfinder.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>interfaces/ibuddydocumentfinder.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>interfaces/idocumentcontroller.h <span style="color: grey">(d036afa)</span></li>

 <li>shell/documentcontroller.h <span style="color: grey">(8dc3706)</span></li>

 <li>shell/documentcontroller.cpp <span style="color: grey">(bc0a3dd)</span></li>

 <li>shell/settings/uiconfig.kcfg <span style="color: grey">(60dccb2)</span></li>

 <li>shell/settings/uiconfig.ui <span style="color: grey">(61f4b08)</span></li>

 <li>shell/tests/CMakeLists.txt <span style="color: grey">(b093565)</span></li>

 <li>shell/tests/documentcontrollertest.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>shell/tests/documentcontrollertest.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>shell/tests/shellbuddytest.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>shell/tests/shellbuddytest.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>sublime/area.h <span style="color: grey">(df0e2bb)</span></li>

 <li>sublime/area.cpp <span style="color: grey">(efa8025)</span></li>

 <li>sublime/controller.h <span style="color: grey">(e8b4794)</span></li>

 <li>sublime/controller.cpp <span style="color: grey">(a8dd38a)</span></li>

 <li>sublime/tests/areaoperationtest.h <span style="color: grey">(0e0f08e)</span></li>

 <li>sublime/tests/areaoperationtest.cpp <span style="color: grey">(28dbeff)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/100874/diff/" style="margin-left: 3em;">View Diff</a></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>

<div>

 <a href="http://git.reviewboard.kde.org/r/100874/s/103/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/03/16/uisettings_400x100.png" style="border: 1px black solid;" alt="New UiSettings" /></a>

</div>


  </td>
 </tr>
</table>




  </div>
 </body>
</html>