Context Navigation

Changes between Version 2 and Version 3 of Ticket #32075, comment 31

-                      v2
+                      v3
 * The `256 * MB_IN_BYTES` is an unnecessary calculation, the end value will never change, so it should be a static value. If this change was made for clarity a `// = 256Mb` comment would be more appropriate.
-Let me know which points you consider valid and I'll happily update the patch later today.
 [Edit]
 Oh and one more thing: the return value can actually be set with more precision as we can check whether the `ini_set()` succeeded.
 That way `false` is returned if nothing was changed and the new limit which was set will be returned if all is well.
+And another: spelling change to `changeable` has not been consistently applied everywhere.
+Last thing out of curiosity - I've been racking my brain what the `hr` in the `wp_convert_hr_to_bytes()` function name stands for... anyone care to enlighten me ? Might be helpful to have as part of the documentation as well.