[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