[Performous-commit] Miguel Sánchez de León Peque : Tags for INI files discussed with FoFiX developers to reach an agreement.

[Peque <ms...@us...>]

Module: performous
Branch: master
Commit: 3c7069f01ee0e4c4a7df512f1a32fd6ce3c8f8ca

Author: Miguel Sánchez de León Peque <peq...@ho...>
Date:   Sun Jan 10 20:02:10 2010 +0100

Tags for INI files discussed with FoFiX developers to reach an agreement.
Now all tags used in songparser-ini.cc are "standard" or at least FoF compatible.

---

 game/songparser-ini.cc |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/game/songparser-ini.cc b/game/songparser-ini.cc
index caa00d2..6ac8c8a 100644
--- a/game/songparser-ini.cc
+++ b/game/songparser-ini.cc
@@ -46,14 +46,12 @@ void SongParser::iniParse() {
 		boost::trim(value);
 		if (key == "name") s.title = value;
 		else if (key == "artist") s.artist = value;
-		// These are added features (not part of FoF, though video is supported in FoFiX SVN)
-		// TODO these tags need to be checked/discussed with FoFiX developers
 		else if (key == "cover") s.cover = value;
 		else if (key == "background") s.background = value;
 		else if (key == "video") s.video = value;
 		else if (key == "video_start_time") { assign(s.videoGap, value); s.videoGap/=1000.0; }
-		else if (key == "prev_start_time") { assign(s.preview_start, value); s.preview_start/=1000.0; }
-		// End of added features
+		else if (key == "preview_start_time") { assign(s.preview_start, value); s.preview_start/=1000.0; }
+		// Before adding other tags: they should be checked with the already-existing tags in FoF format; in case any tag doesn't exist there, it should be discussed with FoFiX developers before adding it here.
 	}
 	if (s.title.empty() || s.artist.empty()) throw std::runtime_error("Required header fields missing");