dlang.org icon indicating copy to clipboard operation
dlang.org copied to clipboard
verified publisher icon dlang

Update c++ interop reference page on lifetime handling interoperability

Open Emmankoko opened this issue 1 year ago • 4 comments

calling C++ constructors in D are now possible and so must be updated in documentation @dkorpel

Emmankoko avatar Apr 03 '24 19:04 Emmankoko

Thanks for your pull request and interest in making D better, @Emmankoko! We are looking forward to reviewing it, and you should be hearing from a maintainer soon. Please verify that your PR follows this checklist:

  • My PR is fully covered with tests (you can see the coverage diff by visiting the details link of the codecov check)
  • My PR is as minimal as possible (smaller, focused PRs are easier to review than big ones)
  • I have provided a detailed rationale explaining my changes
  • New or modified functions have Ddoc comments (with Params: and Returns:)

Please see CONTRIBUTING.md for more information.

If you have addressed all reviews or aren't sure how to proceed, don't hesitate to ping us with a simple comment.

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

dlang-bot avatar Apr 03 '24 19:04 dlang-bot

Okay so I have finally been able to get this PR pass with the CI. I'm sharing the preview of my changes here from CyberShadow. After you have a look, we can discuss some of your changes you might suggest. At the lifetime handling header

https://dtest.dlang.io/artifact/website-a787afecc4e6f5a767e58446ec396334c940e628-1441f51d92dcf19ade5fe29608bbed4d/web/spec/cpp_interface.html

@dkorpel

Emmankoko avatar Apr 04 '24 14:04 Emmankoko

Thanks for writing this! I don't use C++ so I can't verify the info deeply, perhaps @Geod24 can take a quick look?

Okay let me ping him

Emmankoko avatar Apr 05 '24 18:04 Emmankoko

CC @TurkeyMan

Geod24 avatar Sep 18 '24 15:09 Geod24