1
0
mirror of https://github.com/cookiengineer/audacity synced 2025-06-21 14:50:06 +02:00

Correct recent comments...

... compatiblity of plug-in settings is the real problem, not the plug-in
registry.
This commit is contained in:
Paul Licameli 2018-01-16 12:27:17 -05:00
parent 4628d6afa7
commit af279072b5
2 changed files with 3 additions and 3 deletions

View File

@ -1250,7 +1250,7 @@ wxString PluginDescriptor::GetTranslatedEffectFamily() const
#else
// PRL: 2.2.2 hack to change the visible name without breaking
// compatibility of pluginregistry.cfg; redo this better
// compatibility of pluginsettings.cfg; redo this better
// Remap "Audacity" to "Built-in" (suitably translated)
// "Audacity" was the only possibility for mEffectFamily that was in the

View File

@ -222,7 +222,7 @@ wxString Effect::GetFamily()
// PRL: In 2.2.2 we wanted to change the user-visible name to
// "Built-in" but we did not do it the obvious way by just changing this
// string, because of problems with compatibility of pluginregistry.cfg
// string, because of problems with compatibility of pluginsettings.cfg
// See PluginDescriptor::GetTranslatedEffectFamily and
// EffectUIHost::OnMenu
// See PluginManager::RegisterPlugin and PluginManager::GetID, where the
@ -3360,7 +3360,7 @@ void EffectUIHost::OnMenu(wxCommandEvent & WXUNUSED(evt))
auto type = mEffect->GetFamily();
// PRL: 2.2.2 hack to change the visible name without breaking
// compatibility of pluginregistry.cfg; redo this better
// compatibility of pluginsettings.cfg; redo this better
// See also PluginDescriptor::GetTranslatedEffectFamily
if (type == wxT("Audacity"))
type = XO("Built-in");