Skip to content

Fix #14680: GUI: internalError with empty user includes#8478

Merged
ludviggunne merged 4 commits intodanmar:mainfrom
ludviggunne:14680
Apr 23, 2026
Merged

Fix #14680: GUI: internalError with empty user includes#8478
ludviggunne merged 4 commits intodanmar:mainfrom
ludviggunne:14680

Conversation

@ludviggunne
Copy link
Copy Markdown
Collaborator

No description provided.

@firewave
Copy link
Copy Markdown
Collaborator

See also #8475 (superseded by this) and #8462.

@ludviggunne
Copy link
Copy Markdown
Collaborator Author

See also #8475 (superseded by this) and #8462.

Ah... I did think it was strange that nobody apparently had picked up on this, I was only looking at the list of tickets. This PR does add a test though, but maybe it's trivial enough to skip?

@ludviggunne
Copy link
Copy Markdown
Collaborator Author

This PR does add a test though, but maybe it's trivial enough to skip?

And trimming the input it might be good anyway.

Comment thread gui/projectfile.cpp
@sonarqubecloud
Copy link
Copy Markdown

@ludviggunne ludviggunne merged commit 4d12b1d into danmar:main Apr 23, 2026
85 of 87 checks passed
@ludviggunne ludviggunne deleted the 14680 branch April 23, 2026 12:12
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.

3 participants