From 415f9a1b0b510dead1c0c8afa17d42c7065541c0 Mon Sep 17 00:00:00 2001 From: Max Horn Date: Wed, 27 Oct 2010 09:47:18 +0000 Subject: AGI: Add note about AgiEngine::parseFeatures svn-id: r53872 --- engines/agi/agi.cpp | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) (limited to 'engines/agi/agi.cpp') diff --git a/engines/agi/agi.cpp b/engines/agi/agi.cpp index c8689fd9c4..4dcf5fee5e 100644 --- a/engines/agi/agi.cpp +++ b/engines/agi/agi.cpp @@ -715,6 +715,33 @@ Common::Error AgiEngine::go() { } void AgiEngine::parseFeatures() { + + /* FIXME: Seems this method doesn't really do anything. It might + be a leftover that could be removed, except that some of its + intended purpose may still need to be reimplemented. + + [0:29] can you tell me what the point behind AgiEngine::parseFeatures() is? + [0:30] <_sev> when games are created with WAGI studio + [0:31] <_sev> it creates .wag site with game-specific features such as full game title, whether to use AGIMOUSE etc + [0:32] ... and the "features" config key is created by our detector based on the wag file, I guess? + [0:33] <_sev> yes + [0:33] it's just that I cant seem to find a place we do that + [0:33] <_sev> it is used for fallback + [0:34] <_sev> ah, perhaps it was not updated + [0:34] I only see us check the value, but never set it + [0:34] maybe I am grepping wrong, who knows :) + [0:44] _sev: so, unless I miss something, it seem that function does nothing right now + [0:45] <_sev> Fingolfin: it could be unfinished. It was part of GSoC 3 years ago + [0:45] well + [0:45] <_sev> I just don't remember + [0:45] but don't we just re-parse the wag when the game is loaded anyway? + [0:45] <_sev> but it documents the format + [0:45] the advanced meta engine would re-run the detector, wouldn't it? + [0:45] <_sev> yep + [0:47] so... shouldn't we at least add a comment to the function explaining what it does and that it's unfinished etc.? maybe add a TODO to the wiki? + [0:47] otherwise it might stay as it is for another 3 years :) + */ + if (!ConfMan.hasKey("features")) return; -- cgit v1.2.3