Skip to content

Replaced ERR_EXPLAIN with CRASH_NOW_MSG for 3.2.1 compatibility#3

Open
yeslayla wants to merge 2 commits into
perdugames:masterfrom
yeslayla:master
Open

Replaced ERR_EXPLAIN with CRASH_NOW_MSG for 3.2.1 compatibility#3
yeslayla wants to merge 2 commits into
perdugames:masterfrom
yeslayla:master

Conversation

@yeslayla

Copy link
Copy Markdown

Replaced error messaging functions to allow GDNet3 to build for Godot 3.2.1

Related Godot PR: godotengine/godot#33517

@demurzasty

demurzasty commented Jan 23, 2021

Copy link
Copy Markdown

Will it not be better when we use ERR_FAIL_COND_V_MSG and ERR_FAIL_COND_MSG instead of CRASH_NOW_MSG?

@yeslayla

Copy link
Copy Markdown
Author

Will it not be better when we use ERR_FAIL_COND_V_MSG and ERR_FAIL_COND_MSG instead of CRASH_NOW_MSG?

It would probably be a better solution, though this PR is about a year old now, so I'm not sure how stale it is.

@yeslayla

Copy link
Copy Markdown
Author

Updated commit, implements changes suggested by @demurzasty and makes the project compatible with stable and 3.3rc

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