Skip to content

Another pack of improvements to daScript/daScriptBind.h#1567

Merged
borisbat merged 1 commit into
GaijinEntertainment:masterfrom
zheka2304:master
Mar 11, 2025
Merged

Another pack of improvements to daScript/daScriptBind.h#1567
borisbat merged 1 commit into
GaijinEntertainment:masterfrom
zheka2304:master

Conversation

@zheka2304
Copy link
Copy Markdown

  1. Now ADD_FUN/METHOD_BIND will automatically select appropriate simulation node: SimNode_ExtFuncCallRef if result is a reference, SimNode_ExtFuncCall otherwise (In case of returning by value it is still up to the user, attempt to determine this automatically too by using has_cast has shown that it is not viable)
  2. DAS_MODULE_DECL_EX can now accept several include paths, each of them can contain quotes (either " or <) or not (" used by default)
  3. Rework asserting stub, it will now not link copy constructor, and will always crash after assert by calling nullptr instead of returning garbage and possibly continue execution
  4. Fixed DAS_ADD_USING_BIND

Change-Id: I0eeac047b60d72f03ebe9009ae0dd8b9782f0419

1. Now ADD_FUN/METHOD_BIND will automatically select appropriate simulation node: SimNode_ExtFuncCallRef if result is a reference, SimNode_ExtFuncCall otherwise (In case of returning by value it is still up to the user, attempt to determine this automatically too by using has_cast has shown that it is not viable)
2. DAS_MODULE_DECL_EX can now accept several include paths, each of them can contain quotes (either " or <) or not (" used by default)
3. Rework asserting stub, it will now not link copy constructor, and will always crash after assert by calling nullptr instead of returning garbage and possibly continue execution
4. Fixed DAS_ADD_USING_BIND

Change-Id: I0eeac047b60d72f03ebe9009ae0dd8b9782f0419
@borisbat borisbat merged commit 76b45c8 into GaijinEntertainment:master Mar 11, 2025
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