fuzzie has covered what I was about to say already, but I'll have to add my 2c here.<br><br>In my eyes, this is an undocumented (and annoying) feature that is there just to cancel another feature (game IDs). I wrote the comment this way because this code bypasses a whole feature on purpose, without even specifying what it does... I was pulling my hair trying to figure out why kyra is returning a game ID in the advanced detector but SCI isn't (which is bad, common code should be documented). The fact that this code has been around for long does not mean it works, it is a workaround (at least, silently changing a feature has been considered a workaround historically).<br>
<br>I did not change any functionality in the common code, I just added a comment about why this functionality offered by singleId is not ideal, at least not IMHO, since it just changes a default behavior and interferes with other common code. And worst of all, this wasn't documented anywhere, hence my commit and subsequent revert.<br>
<br>As for a discussion: this was code that was just committed in the past, and I don't recall why it was added and if there was a discussion concerning it in the first place (at least I couldn't find any...).<br>
<br>Regards<br>Filippos<br><br><div class="gmail_quote">On Tue, Feb 28, 2012 at 11:17 PM, Alyssa Milburn <span dir="ltr"><<a href="mailto:reply%2Bc-1023204-5c06e562b6391148f90d33579bf4201a13427d98-510395@reply.github.com">reply+c-1023204-5c06e562b6391148f90d33579bf4201a13427d98-510395@reply.github.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Silently replacing the detected gameid with the singleid is an ugly workaround which is completely surprising in behaviour and was still completely undocumented. Searching for 'singleid' on the mailing list doesn't produce any results. If one of you has better documentation, please add it.<br>

<br>
I expressed surprise at [md5]'s SCI commit earlier, which was obviously in reaction to discovery that the detector was overwriting the gameid information with this weird 'singleid' thing, and [md5] came on IRC and agreed to revert the commit until a better solution was found for identifying the games which configuration entries correspond to without having to re-run the detector.<br>

<br>
I explained that I thought some of the aim of this was to avoid polluting the global namespace, and so simply removing singleid was a bad idea. If that isn't the case, and we can simply remove singleid from engines as we wish, then obviously that simplifies things. But I don't think it's fair to act surprised at confusion over this completely undocumented and annoying "feature".<br>

<div class="im"><br>
---<br>
Reply to this email directly or view it on GitHub:<br>
</div><a href="https://github.com/scummvm/scummvm/commit/65c60ef02737f9e6a7e4091b622a6a1e0871d6d7#commitcomment-1023204" target="_blank">https://github.com/scummvm/scummvm/commit/65c60ef02737f9e6a7e4091b622a6a1e0871d6d7#commitcomment-1023204</a><br>

</blockquote></div><br>