[whups] Re: [hermes] Making clients
Jason M. Felice
jfelice at cronosys.com
Wed Jul 16 09:28:08 PDT 2003
On Wed, Jul 16, 2003 at 06:59:01PM +1000, Bo Daley wrote:
> does this make any sense? comments? etc?
Here's my comments on the patch:
* Why the 'list' API in whups? You can get pretty much the same thing with
'search' with the $names arg set to array('').
* Never mind about the deleteLink() issue, I had my links driver set to
none. I've submitted a patch to provide a deleteLink() stub for the
'none' driver, and set up the sql driver.
* Coding standard: use "One True Brace", e.g.:
function foo()
{
It looks good!
--
Jason M. Felice
Cronosys, LLC <http://www.cronosys.com/>
216.221.4600 x302
More information about the whups
mailing list