<br><br><div class="gmail_quote">On Wed, Apr 11, 2012 at 2:53 PM, Caolán McNamara <span dir="ltr">&lt;<a href="mailto:caolanm@redhat.com">caolanm@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Tue, 2012-04-10 at 12:52 +0200, Lubos Lunak wrote:<br>
&gt;      List*                   GetSelectEntryList( sal_uInt16 nDepth );<br>
&gt; +    void                    GetSelectEntryList (sal_uInt16 nDepth,<br>
&gt; std::vector&lt;rtl::OUString&gt; &amp;rEntries) const;<br>
&gt;<br>
&gt;  Why is that? Changing the return value to a reference argument makes the API<br>
&gt; worse and it seems like an unnecessary change to me.<br>
<br>
</div>Sorry, missed your earlier review. Yeah, struck me as well, but a quick<br>
attempt to remove all uses of the older one shows that there is a good<br>
bit of work still to do before e.g. removing the older<br>
GetSelectEntryList and making the new one return a vector, so the two<br>
still need to coexist for the moment. Maybe renaming the new<br>
GetSelectEntryList to say, getSelectEntryList, and have it return a<br>
vector, or let Rafael iterate through the rest of the nasty cases and<br>
change over afterwards.<br>
<span class="HOEnZb"><font color="#888888"><br>
C.<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org">LibreOffice@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
</div></div></blockquote></div><br>Well the first reason is that you cant overload a function with a return value, second reason and the main one, is that copying a vector is &quot;costly&quot; so to prevent that i passes it by reference avoids that, but i can change it later if needed. I already finished the porting all the needed code, but didnt want to send it all because its alot of patches. I wanted another people review to double check i didnt mess it up.<br>
<br>Can i send the rest of the patches? or maybe send it by parts?? Thanks for reviewing <br>