lee/renderdoc
1
0
Fork 0
mirror of https://github.com/baldurk/renderdoc.git synced 2026-09-08 14:55:41 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix problem where a new profile wouldn't have default event filter

Browse Source 
* In v1.15 if you upgraded from a previous version and had a renderdoc config
  file then the persistent storage storing the filters would get loaded with an
  empty variant map, so the defaults could be set. However if you had no config
  file at all this wouldn't happen so no defaults would be set.
* We detect the case where we load an old profile and it has blank current
  filter or saved filters, and fill in the defaults now. If the filter has been
  customised or some saved filters exist, respectively, we don't touch them.
This commit is contained in:
baldurk
2021-08-24 10:57:36 +01:00
parent 9486cac1ca
commit f8cf943c42
1 changed files with 48 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
qrenderdoc/Windows/EventBrowser.cpp
+48 -6
View File
@@ -55,12 +55,25 @@
struct EventBrowserPersistentStorage : public CustomPersistentStorage
{
EventBrowserPersistentStorage() : CustomPersistentStorage(rdcstr()) {}
EventBrowserPersistentStorage(rdcstr name) : CustomPersistentStorage(name) {}
EventBrowserPersistentStorage() : CustomPersistentStorage(rdcstr())
{
version = LatestVersion;
SetDefaultCurrentFilter();
AddDefaultSavedFilters();
}
EventBrowserPersistentStorage(rdcstr name) : CustomPersistentStorage(name)
{
version = LatestVersion;
SetDefaultCurrentFilter();
AddDefaultSavedFilters();
}
void save(QVariant &v) const
{
QVariantMap settings;
settings[lit("version")] = LatestVersion;
settings[lit("current")] = CurrentFilter;
QVariantList filters;
@@ -76,6 +89,8 @@ struct EventBrowserPersistentStorage : public CustomPersistentStorage
{
QVariantMap settings = v.toMap();
int loadedVersion = settings[lit("version")].toInt();
QVariant current = settings[lit("current")];
if(current.isValid() && current.type() == QVariant::String)
{
@@ -83,9 +98,11 @@ struct EventBrowserPersistentStorage : public CustomPersistentStorage
}
else
{
CurrentFilter = lit("$action()");
SetDefaultCurrentFilter();
}
SavedFilters.clear();
QVariant saved = settings[lit("filters")];
if(saved.isValid() && saved.type() == QVariant::List)
{
@@ -106,12 +123,37 @@ struct EventBrowserPersistentStorage : public CustomPersistentStorage
}
else
{
SavedFilters.push_back(qMakePair(lit("Default"), lit("$action()")));
SavedFilters.push_back(qMakePair(lit("Actions and Barriers"), lit("$action() Barrier")));
SavedFilters.push_back(qMakePair(lit("Hide Copies & Clears"), lit("$action() -Copy -Clear")));
AddDefaultSavedFilters();
}
// version 2 we fixed an issue where a default profile wouldn't have the proper default filter
// etc. That shipped in v1.15.
// If we detect an old profile being loaded AND there are: no saved filters, or empty filter, we
// choose to override and set the default filter/add saved filters. If the user has set a
// filter, or saved some, we don't do anything as they have used the feature and we'll respect
// what they've done. If the user has deliberately left it blank we have no way of telling, so
// we set the default filter to cover the common case of someone who hasn't used the filter at
// all.
if(loadedVersion < 2)
{
if(CurrentFilter.isEmpty())
SetDefaultCurrentFilter();
if(SavedFilters.isEmpty())
AddDefaultSavedFilters();
}
}
void SetDefaultCurrentFilter() { CurrentFilter = lit("$action()"); }
void AddDefaultSavedFilters()
{
SavedFilters.push_back(qMakePair(lit("Default"), lit("$action()")));
SavedFilters.push_back(qMakePair(lit("Actions and Barriers"), lit("$action() Barrier")));
SavedFilters.push_back(qMakePair(lit("Hide Copies & Clears"), lit("$action() -Copy -Clear")));
}
static const int LatestVersion = 2;
int version;
QString CurrentFilter;
QList<QPair<QString, QString>> SavedFilters;
};