<div dir="ltr">Looks like a copy and paste error. I'd prefer to have the function removed in both trunk and the 1.1.0 branch. <div><br></div><div>Yotam<br><br><div class="gmail_quote">On Thu, Mar 18, 2010 at 10:22 PM, Max Horn <span dir="ltr"><<a href="mailto:max@quendi.de">max@quendi.de</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi folks,<br>
<br>
I am afraid some of my commits today broke compilation of the DC and PSP ports. I since then fixed this, but the reason for the compile errors is actually a potential issue / bug in the underlying code, so it's best if you guys review this:<br>


<br>
Basically, both the DC and PSP backends subclass FilePluginProvider, and overload the addCustomDirectories method. However, in the base class, its signature is:<br>
<br>
  virtual void addCustomDirectories(Common::FSList &dirs) const;<br>
<br>
while in your subclasses, it was<br>
<br>
  virtual void addCustomDirectories(Common::StringList &dirs) const;<br>
<br>
Since I renamed StringList to StringArray today, and moved its definition, this caused compile errors in your files. But the actual issue at hand is clearly a mismatch in signature; your custom methods were never being called.<br>


<br>
I changed the signatures of the functions, and adjusted the code to make it compile. However, I cannot verify that it works now, e.g. I don't know whether your FSNode's handle "/". So, please test and review this. See also <<a href="http://scummvm.svn.sourceforge.net/scummvm/?rev=48290&view=rev" target="_blank">http://scummvm.svn.sourceforge.net/scummvm/?rev=48290&view=rev</a>><br>


<br>
<br>
Extra warning: This issue also exists in the 1.1.0 branch, i.e. you have addCustomDirectories methods which are not being called. So I guess it should be either fixed there, too, or else the custom addCustomDirectories should be removed in both branch and trunk.<br>


<br>
<br>
Cheers,<br>
<font color="#888888">Max</font></blockquote></div><br></div></div>