Re: [PECL-DEV] Re: [PHP-DOC] PECL/Memcache documentation
[email protected] (Brian Moon) Thu, 24 Jun 2010 09:30:17 -0700
| Newsgroups | php.doc,php.pecl.dev |
|---|---|
| Message-ID | <[email protected]> |
> The default $timeout sent over to memcached, if none was specified, is > zero (is it?). If that is the case, the docs need to reflect that > (even if the value isn't/shouldn't ever be used) with something like > ``...<parameter>timeout</parameter><initializer>0</initializer></methodparam>`` > -- of course, if there is no default value, then there is no need for > an initializer element. So, just to confirm, the existing docs are wrong and you need to me to fix them before any other changes can be made? That sounds curt, not meant to be. Just want to understand what needs to happen to get this done. >>> And lastly, changes in behavour should be described in the<changelog> >>> role within the methods docs. >> >> And here you really lost me. I have not done PHP doc work in 10 years so >> bear with me. I am sure if you work on them every day this makes sense, but >> it reads like Java to me. ;) > > Presumably, the $timeout parameter worked at some point? The > changeover from being accepted to being deprecated/unsupported needs > to be documented in a "Changelog" block[1] so that folks using an old > version (god forbid) can see how things were rather than how things > are. No, it never worked. The extension was wrong wrong wrong always. It lied to the user about what the timeout did. Yes, memcached took the parameter in the past, but never with the behavior that was documented here. But, I will do whatever it takes to get these docs updated. Brian.