Skip to content
Snippets Groups Projects
Commit 0da8fb60 authored by Kent Quirk's avatar Kent Quirk
Browse files

Change comment that wasn't updated with the code. No functionality changes.

parent 4ed913c3
No related branches found
No related tags found
No related merge requests found
......@@ -352,20 +352,18 @@ void LLLoginInstance::updateApp(bool mandatory, const std::string& auth_msg)
payload["mandatory"] = mandatory;
/*
We're constructing one of the following 6 strings here:
We're constructing one of the following 9 strings here:
"DownloadWindowsMandatory"
"DownloadWindowsReleaseForDownload"
"DownloadWindows"
"DownloadMacMandatory"
"DownloadMacReleaseForDownload"
"DownloadMac"
"DownloadLinuxMandatory"
"DownloadLinuxReleaseForDownload"
"DownloadLinux"
I've called them out explicitly in this comment so that they can be grepped for.
Also, we assume that if we're not Windows we're Mac. If we ever intend to support
Linux with autoupdate, this should be an explicit #elif LL_DARWIN, but
we'd rather deliver the wrong message than no message, so until Linux is supported
we'll leave it alone.
*/
std::string notification_name = "Download";
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment