actions/itgmania212121
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

23b02ce: fix BackgroundChange color values being written with commas. Commas are a reserved character that seprate values in a tag. Caret was used to separate RGBA values in a color in this case and NotesLoaderSM already handles that. [Chris Danford]

Browse Source
This commit is contained in:
AJ Kelly
2010-06-08 18:16:54 -05:00
parent b0da1de6db
commit f8a9423301
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/NotesWriterSM.cpp
+10 -2
View File
@@ -16,6 +16,14 @@
#include "Song.h" #include "Song.h"
#include "Steps.h" #include "Steps.h"
static RString EscapeBackgroundColor( const RString color )
{
// Comma separates values in a background change. Caret separates RGBA values in a color value, and NotesLoaderSM switches them to commas on load.
RString c = color;
c.Replace(",", "^");
return c;
}
static RString BackgroundChangeToString( const BackgroundChange &bgc ) static RString BackgroundChangeToString( const BackgroundChange &bgc )
{ {
// TODO: Technically we need to double-escape the filename (because it might contain '=') and then // TODO: Technically we need to double-escape the filename (because it might contain '=') and then
@@ -31,8 +39,8 @@ static RString BackgroundChangeToString( const BackgroundChange &bgc )
bgc.m_def.m_sEffect.c_str(), bgc.m_def.m_sEffect.c_str(),
bgc.m_def.m_sFile2.c_str(), bgc.m_def.m_sFile2.c_str(),
bgc.m_sTransition.c_str(), bgc.m_sTransition.c_str(),
SmEscape(bgc.m_def.m_sColor1).c_str(), SmEscape(EscapeBackgroundColor(bgc.m_def.m_sColor1)).c_str(),
SmEscape(bgc.m_def.m_sColor2).c_str() SmEscape(EscapeBackgroundColor(bgc.m_def.m_sColor2)).c_str()
); );
return s; return s;
} }