Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Preview 1.82.18 Hotfix #690

Merged
merged 21 commits into from
Feb 19, 2025
Merged

Preview 1.82.18 Hotfix #690

merged 21 commits into from
Feb 19, 2025

Conversation

bagusnl
Copy link
Member

@bagusnl bagusnl commented Feb 16, 2025

What's changed? - 1.82.18

  • [Fix] Launcher crashing when the main game thread unexpectedly closes, by @neon-nyan
  • [Fix] Taskbar progress not reset after game install/update, by @shatyuka
  • [Fix] Ensures temporary files availability, by @neon-nyan
  • [Fix] 7z COMException errors, by @neon-nyan
  • [Fix] Genshin's repair missing a few files, by @neon-nyan

Templates

Changelog Prefixes
  **[New]**
  **[Imp]**
  **[Fix]**
  **[Loc]**
  **[Doc]**

Copy link

github-actions bot commented Feb 16, 2025

Qodana for .NET

10 new problems were found

Inspection name Severity Problems
Possible 'System.NullReferenceException' 🔶 Warning 4
Possible null reference argument for a parameter. 🔶 Warning 2
Unused local variable 🔶 Warning 2
Conditional access qualifier expression is not null according to nullable reference types' annotations 🔶 Warning 1
Non-accessed positional property (non-private accessibility) 🔶 Warning 1

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

// If the ignoreContainsParams is not null and the remoteName contains
// ignore list, then move to another entry
if (ignoreContainsParams != null &&
manifestEntry.remoteName

Check warning

Code scanning / QDNET

Possible 'System.NullReferenceException' Warning

Possible 'System.NullReferenceException'
}

// Resolve the svc_catalog name where the localName contains "../"
if (manifestEntry.remoteName.EndsWith("svc_catalog"))

Check warning

Code scanning / QDNET

Possible 'System.NullReferenceException' Warning

Possible 'System.NullReferenceException'
{
entry!.remoteName = $"{parentPath.Replace('\\', '/')}/{entry.remoteName}";
}
string relativeParentPath = string.IsNullOrEmpty(parentPath) ? "" : GetParentFromAssetRelativePath(entry.remoteName, out _);

Check warning

Code scanning / QDNET

Possible 'System.NullReferenceException' Warning

Possible 'System.NullReferenceException'
entry!.remoteName = $"{parentPath.Replace('\\', '/')}/{entry.remoteName}";
}
string relativeParentPath = string.IsNullOrEmpty(parentPath) ? "" : GetParentFromAssetRelativePath(entry.remoteName, out _);
entry.remoteName = Path.Combine(parentPath ?? "", relativeParentPath, entry.remoteName);

Check warning

Code scanning / QDNET

Possible 'System.NullReferenceException' Warning

Possible 'System.NullReferenceException'
@Cryotechnic
Copy link
Collaborator

Approved via VC

@Cryotechnic Cryotechnic merged commit cddd41b into preview Feb 19, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants