updated comments

This commit is contained in:
David Kelly 2017-06-22 07:09:24 -07:00
parent 92cbb86e8b
commit 5da01f0074
2 changed files with 9 additions and 5 deletions

View file

@ -859,11 +859,14 @@ Application::Application(int& argc, char** argv, QElapsedTimer& startupTimer, bo
}
ResourceCache::setRequestLimit(concurrentDownloads);
// perhaps override the avatar url. We test later for validity, so
// no need to do so here. If someone specifies both --avatarURL
// and --replaceAvatarURL, the replaceAvatarURL wins.
// perhaps override the avatar url. Since we will test later for validity
// we don't need to do so here.
QString avatarURL = getCmdOption(argc, constArgv, "--avatarURL");
_avatarOverrideUrl = QUrl::fromUserInput(avatarURL);
// If someone specifies both --avatarURL and --replaceAvatarURL,
// the replaceAvatarURL wins. So only set the _overrideUrl if this
// does have a non-empty string.
QString replaceURL = getCmdOption(argc, constArgv, "--replaceAvatarURL");
if (!replaceURL.isEmpty()) {
_avatarOverrideUrl = QUrl::fromUserInput(replaceURL);

View file

@ -929,8 +929,9 @@ void MyAvatar::saveData() {
settings.setValue("scale", _targetScale);
// only save the fullAvatarURL if it has not been overwritten on command line, or it
// has but we want it saved
// only save the fullAvatarURL if it has not been overwritten on command line
// (so the overrideURL is not valid), or it was overridden _and_ we specified
// --replaceAvatarURL (so _saveAvatarOverrideUrl is true)
if (qApp->getSaveAvatarOverrideUrl() || !qApp->getAvatarOverrideUrl().isValid() ) {
settings.setValue("fullAvatarURL",
_fullAvatarURLFromPreferences == AvatarData::defaultFullAvatarModelUrl() ?