<br><br><div class="gmail_quote">On Tue, Feb 22, 2011 at 1:35 PM, Jan Holesovsky <span dir="ltr">&lt;<a href="mailto:kendy@suse.cz">kendy@suse.cz</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi Joseph,<br>
<div class="im"><br>
On 2011-02-20 at 20:40 -0800, Joseph Powers wrote:<br>
&gt; Here&#39;s a quick update on the project:<br>
&gt;<br>
&gt; 1. The only items left are:<br>
<br>
</div>Very nice! :-)<br>
<div class="im"><br>
&gt; 2. We also have a few locations where the &quot;List&quot; class is used<br>
&gt; directly with out the DECLARE_LIST() macro.<br>
&gt;<br>
&gt;<br>
&gt; My work order is:<br>
&gt;<br>
&gt;<br>
&gt; 1. Reduce the number of #include &quot;tools/list.hxx&quot; down to a minimum<br>
&gt; (it should only be used in areas that use the List class).<br>
&gt; 2. Convert the binfilter &amp; svl items above to a ::std::vector&lt;&gt; or<br>
&gt; similar template. These files are only used on MS-Windows which I<br>
&gt; don&#39;t build on, so I&#39;m going to just create the patches and then send<br>
&gt; them to Fridrich who volunteered to make sure the build ok.<br>
<br>
</div>My take would be - don&#39;t care about binfilter that much ;-)  If the code<br>
other than binfilter compiles without eg. DECLARE_LIST (or the List<br>
type), I&#39;d just move the macro (or the List type) from tools/ to a<br>
header that is internal for binfilter, and let it be (for now); it might<br>
very much turn out that it will be deleted by the removal of write<br>
support in binfilter.<br>
<br>
[But of course, if you want to get rid of that for good, go for that.]<br>
<div class="im"><br>
&gt; 3. Fix the tools/inc/tools/ref.hxx usage of DECLARE_LIST. The issue<br>
&gt; here is that it&#39;s a macro that&#39;s used in several locations.<br>
&gt;<br>
&gt; Rafael is working on the following areas:<br>
&gt;<br>
&gt; 1. Convert the idl items above.<br>
&gt; 2. Start converting the code that uses the List class.<br>
<br>
</div>Cool!<br>
<br>
[BTW, another list-like thing that I&#39;ve just spotted:<br>
<a href="http://opengrok.libreoffice.org/xref/libs-gui/vcl/inc/list.h" target="_blank">http://opengrok.libreoffice.org/xref/libs-gui/vcl/inc/list.h</a> ]<br>
<br>
Thank you a lot, this rocks, :-)<br>
Kendy<br>
<br>
<br>
</blockquote></div>lol how many list implementations they got in the code?? xD talking about duplicating their efforts =P<br><br>