-
-
Notifications
You must be signed in to change notification settings - Fork 553
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
get rid of one ParentWithGens #36893
Conversation
Documentation preview for this PR (built with commit 086a13f; changes) is ready! 🎉 |
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.
Goodbye, you will not be missed.
However, I think we should be a little careful here. Have you checked that the compiled C code just does a C-level redirection? Otherwise we might be some speed regressions.
Indeed, I am probably not careful enough. I am not able to check anything about the generated code, sorry. |
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.
I checked and it should be the same as before:
__pyx_t_1 = __Pyx_GetItemInt(__pyx_v_self->__pyx_base.__pyx_base.__pyx_base.__pyx_base.__pyx_base._parent->__pyx_base._names, 0, long, 1, __Pyx_PyInt_From_long, 0, 0, 1); if (unlikely(!__pyx_t_1)) __PYX_ERR(0, 2342, __pyx_L1_error)
__Pyx_GOTREF(__pyx_t_1);
__pyx_t_6 = PyObject_Hash(__pyx_t_1); if (unlikely(__pyx_t_6 == ((Py_hash_t)-1))) __PYX_ERR(0, 2342, __pyx_L1_error)
__Pyx_DECREF(__pyx_t_1); __pyx_t_1 = 0;
__pyx_v_var_name_hash = __pyx_t_6;
versus before
__pyx_t_1 = __Pyx_GetItemInt(((struct __pyx_obj_4sage_9structure_11parent_gens_ParentWithGens *)__pyx_v_self->__pyx_base.__pyx_base.__pyx_base.__pyx_base.__pyx_base._parent)->__pyx_base.__pyx_base.__pyx_base.__pyx_base._names, 0, long, 1, __Pyx_PyInt_From_long, 0, 0, 1); if (unlikely(!__pyx_t_1)) __PYX_ERR(0, 2343, __pyx_L1_error)
__Pyx_GOTREF(__pyx_t_1);
__pyx_t_4 = PyObject_Hash(__pyx_t_1); if (unlikely(__pyx_t_4 == ((Py_hash_t)-1))) __PYX_ERR(0, 2343, __pyx_L1_error)
__Pyx_DECREF(__pyx_t_1); __pyx_t_1 = 0;
__pyx_v_var_name_hash = __pyx_t_4;
So positive review.
sagemathgh-36893: get rid of one ParentWithGens mirco-ticket to get rid of one `ParentWithGens` ### 📝 Checklist - [x] The title is concise, informative, and self-explanatory. - [x] The description explains in detail what this PR is about. URL: sagemath#36893 Reported by: Frédéric Chapoton Reviewer(s): Travis Scrimshaw
sagemathgh-36893: get rid of one ParentWithGens mirco-ticket to get rid of one `ParentWithGens` ### 📝 Checklist - [x] The title is concise, informative, and self-explanatory. - [x] The description explains in detail what this PR is about. URL: sagemath#36893 Reported by: Frédéric Chapoton Reviewer(s): Travis Scrimshaw
sagemathgh-36893: get rid of one ParentWithGens mirco-ticket to get rid of one `ParentWithGens` ### 📝 Checklist - [x] The title is concise, informative, and self-explanatory. - [x] The description explains in detail what this PR is about. URL: sagemath#36893 Reported by: Frédéric Chapoton Reviewer(s): Travis Scrimshaw
sagemathgh-36893: get rid of one ParentWithGens mirco-ticket to get rid of one `ParentWithGens` ### 📝 Checklist - [x] The title is concise, informative, and self-explanatory. - [x] The description explains in detail what this PR is about. URL: sagemath#36893 Reported by: Frédéric Chapoton Reviewer(s): Travis Scrimshaw
mirco-ticket to get rid of one
ParentWithGens
📝 Checklist