-
Notifications
You must be signed in to change notification settings - Fork 0
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
V3 actual consistent naming #1
Conversation
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
bblfsh/pyuast.cc
Outdated
return node->ctx->Iterate(obj, (TreeOrder)order); | ||
} | ||
// Python object -> create a new context and attach it to an iterator | ||
Context* ctx = new Context(); | ||
return ctx->Iterate(obj, (TreeOrder)order, true); | ||
} | ||
|
||
static PyObject *PyUastDecode(PyObject *self, PyObject *args) { | ||
static PyObject *PyContextDecode(PyObject *self, PyObject *args) { |
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.
Py_Decode
, since it's a global method, or PyContextExt_decode
since it's a constructor specifically for PyContextExt
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 is a perfect example how we could benefit from new github feature :) https://blog.github.com/category/all/#suggested-changes-public-beta
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.
addressed in 2785a60
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.
Looks good, thanks! :)
🎉 |
Refactoring for clarity and consistency, as discussed IRL:
Part of the bblfsh#128