Posted by rimian on August 26, 2010 at 3:07am
2 followers
Jump to:
| Project: | jQuery UI |
| Version: | 6.x-1.3 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | postponed (maintainer needs more info) |
Issue Summary
Hi,
I've written a patch that removes the varying constant and replaces it with a get function. This doesn't change the functionality of this module but does make it easier to read and in line with best practice.
Cheers,
Rim
| Attachment | Size |
|---|---|
| jquery_no_variable_constant.patch | 1.24 KB |
Comments
#1
Why?
#2
Because a constant is supposed to be a constant! If it's the result of an expression then it's variable.