[PATCH] Remove nTableSize from HashTable?
| From: | Matt Wilmas | Date: | Sun, 15 Mar 2009 16:22:06 +0000 |
| Subject: | [PATCH] Remove nTableSize from HashTable? | ||
| Groups: | php.internals | ||
| Request: | Send a blank email to internals+get-43352@lists.php.net to get a copy of this message | ||
Hi Dmitry, all,
Just a quick one... ;^) Is there much, if any, advantage to removing the HashTable member nTableSize? It's just nTableMask + 1, so it can be derived easily in the couple places that need it... Figured it can't hurt to make things smaller, and it only adds a couple addition operations for resizing, basically. :-)
Patches if you want the change (or tell me to commit):
http://realplain.com/php/tablesize.diff
http://realplain.com/php/tablesize_5_3.diff
- Matt