Skip to content

Small Cleanup in UndertaleModTool/Windows/SearchInCodeWindow.xaml.cs#2334

Open
BioTomateDE wants to merge 6 commits into
UnderminersTeam:masterfrom
BioTomateDE:cleanup1
Open

Small Cleanup in UndertaleModTool/Windows/SearchInCodeWindow.xaml.cs#2334
BioTomateDE wants to merge 6 commits into
UnderminersTeam:masterfrom
BioTomateDE:cleanup1

Conversation

@BioTomateDE
Copy link
Copy Markdown
Contributor

Description

error message when text is too long instead of silently returning.
got rid of ugly long tenary expressions
simplified internal GetWordEnding
slightly better error message (more coherent sentence)

Caveats

no im perfect

Notes

it compiles

i fucked up my git branches, the "Improve BuiltinList.cs" pr is also included now lol
sorry

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 12, 2026

@colinator27 colinator27 added this to the Possible for 0.9.x milestone Apr 14, 2026
@colinator27
Copy link
Copy Markdown
Member

I think this will need to be updated to no longer include the changes to BuiltinList.cs. (and merged in general to be up to date)

@BioTomateDE
Copy link
Copy Markdown
Contributor Author

ok i merged it now ;3

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