<blockquote><p dir="ltr"><br>
On 23-Sep-2015 5:37 am, "Aleix Pol Gonzalez" <<a href="mailto:aleixpol@kde.org">aleixpol@kde.org</a>> wrote:<br>
><br>
> This is an automatically generated e-mail. To reply, visit: <a href="https://git.reviewboard.kde.org/r/125353/">https://git.reviewboard.kde.org/r/125353/</a><br>
><br>
> At least do sort once outside of the loop no?<br>
></p>
<p dir="ltr">Ya doing  it outside the loop makes more sense ,will do this . <br><br><br></p>
<p dir="ltr">> Also, are we sure this makes sense at all? Isn't it possible to just click the view's header?<br>
><br>
</p>
</blockquote>
<blockquote><p dir="ltr"><br>
Could you please elaborate more on this ?I didn't understand  what you meant  by view's header ?</p>
<p dir="ltr">><br>
> - Aleix Pol Gonzalez<br>
><br>
><br>
> On September 22nd, 2015, 9:39 p.m. CEST, rishabh gupta wrote:<br>
><br>
> Review request for KDevelop.<br>
> By rishabh gupta.<br>
><br>
> Updated Sept. 22, 2015, 9:39 p.m.<br>
><br>
> Bugs: 119929<br>
> Repository: kdevelop<br>
> Description<br>
><br>
> Bug 119929. after applying the patch variables are listed in the alphabetical order<br>
><br>
> Testing<br>
><br>
> built <br>
> testing done<br>
><br>
> Diffs<br>
> debuggers/gdb/variablecontroller.cpp (33c7353)<br>
><br>
> View Diff</p>
</blockquote>