New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: remove native_mate (Part 5) #20264
Conversation
// | ||
// It also uses gin_helper::CreateFunctionTemplate for function templates to | ||
// support gin_helper types. | ||
class ObjectTemplateBuilder { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe as a follow up we can add this constructor to upstream gin, so that class can reused without duplicating ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This class also relies on our fork of function_template.h
so simply patching gin would not be enough, I'll add comment for it.
@zcbenz this needs rebase then we should be g2g :) |
13cb769
to
ff950c7
Compare
No Release Notes |
Description of Change
Refs #16443.
This PR removes some uses of
mate::ObjectTemplateBuilder
, which is a major blocker to convert class methods togin
.More uses will be removed in the next PR.
Release Notes
Notes: no-notes