Skip to content

Conversation

@ds5678
Copy link
Contributor

@ds5678 ds5678 commented May 6, 2025

Problem

TryOpenStream can return null, and we previously didn't check for that.

Solution

  • Any comments on the approach taken, its consistency with surrounding code, etc.
    • It's similar to the null checks for miscellaneous files.
  • Which part of this PR is most in need of attention/improvement?
  • At least one test covering the code changed

@dgrunwald dgrunwald merged commit 8e7e018 into icsharpcode:master May 7, 2025
5 checks passed
@ds5678 ds5678 deleted the fix-null-reference-exception-when-writing-resource-files branch May 26, 2025 21:59
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.

2 participants