Skip to content

Improve CreateNative failure message#903

Merged
asherkin merged 3 commits intomasterfrom
asherkin-patch-1
Oct 10, 2018
Merged

Improve CreateNative failure message#903
asherkin merged 3 commits intomasterfrom
asherkin-patch-1

Conversation

@asherkin
Copy link
Member

@asherkin asherkin commented Oct 9, 2018

This confuses everyone.

This confuses everyone.
@KyleSanderson
Copy link
Member

Docs are definitely an improvement. Do we care that we call these Fake Natives internally; but Dynamic Natives to pawn? Anyone digging through code will know them by the former name.

@asherkin
Copy link
Member Author

asherkin commented Oct 9, 2018

I belive the intention is that they become just "natives" in the future, and the term "fake" has confused people in the past, I figured "dynamic" was clear but happy to change/remove. I think dropping the qualifier would make the most sense if anything.

@KyleSanderson
Copy link
Member

Yeah; if Fake/Dynamic are ephemeral ("") qualifiers let's drop them entirely to userspace.

@peace-maker
Copy link
Member

You could even go as far as adding the name of the plugin, that has the native already registered in the second error.

Copy link
Member

@Headline Headline left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good improvement, cool with taking as-is, but I also like Peace-Maker's suggestion at identifying the other plugin's name so debugging is easier so if you want to go the extra mile then go for it, otherwise no problem

@asherkin asherkin merged commit 7dd733c into master Oct 10, 2018
@Headline Headline deleted the asherkin-patch-1 branch October 10, 2018 18:09
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.

4 participants