audacia/audacia
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Better resolution than r13716

Browse Source 
The hairbrained "solution" I came up with in r13716
"worked" but it was simply unnecessary and this is
what I should have just done to begin with.

The previous "solution" actually caused some display
issues in the menus.
This commit is contained in:
lllucius 2014-12-08 03:11:20 +00:00
parent 0c51333407
commit 814ed57a68
13 changed files with 14 additions and 37 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/effects/DtmfGen.h
View File

@ -48,7 +48,7 @@ class EffectDtmf : public Generator {
}
virtual wxString GetEffectIdentifier() {
return wxString(wxT("DTMFTone"));
return wxString(wxT("DTMF Tones"));
}
virtual wxString GetEffectDescription() {

25
src/effects/Effect.cpp
View File

@ -175,30 +175,7 @@ wxString Effect::GetName()
return mClient->GetName();
}
// Break up identifier into words based on caps
//
// EffectManager::GetEffectIdentifier() CamelCases identifiers based on
// blank delimited words. But not all legacy effects have blank delimted
// words and were manually CamelCased. So use that to break up the ident
// into words
//
// This is only temporary and will go away when the legacy effects
// get converted.
//
wxString ident = GetEffectIdentifier();
wxString name;
for (size_t i = 0, cnt = ident.Len(); i < cnt; i++)
{
wxChar c = ident.GetChar(i);
if (wxIsupper(c) && i != 0)
{
name += wxT(' ');
}
name += c;
}
return name;
return GetEffectIdentifier();
}
wxString Effect::GetVendor()