#19642: django.utils.functional.lazy issues
----------------------------------+--------------------------------------
Reporter: void | Owner: aaugustin
Type: Bug | Status: closed
Component: Core (Other) | Version: master
Severity: Normal | Resolution: wontfix
Keywords: lazy, functional | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
----------------------------------+--------------------------------------
Changes (by aaugustin):
* status: assigned => closed
* resolution: => wontfix
Comment:
The second item was fixed in the patch for #19160. The three others are
fairly minor and don't have actual consequences in the current code base.
The patch fixes a few things but it isn't quite production-ready
(`builtin_method_names = ['__hash__', '__str__']` - ouch)
For these reasons I don't think it's useful to keep this ticket open.
----
However, as discussed on IRC, it could be a good idea to re-implement
`lazy()` by overriding `__getattribute__` rather than by copying methods
around, both design-wise and performance-wise (to be benchmarked). Feel
free to write a patch for this and open a new ticket!
--
Ticket URL: <https://code.djangoproject.com/ticket/19642#comment:5>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
