[whups] Re: [hermes] Making clients
Bo Daley
bo at tilda.com.au
Wed Jul 16 19:23:11 PDT 2003
Quoting "Jason M. Felice" <jfelice at cronosys.com>:
> 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('').
Yep I agree that it would be better done without any change to the Turba API
(given that the 'search' method already effectively provides the list function
anyway. I did try that initially but got a bit lost along the way. I'll have
another go at it now.
> * 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.
cool -- thanks for finding that one.
> * Coding standard: use "One True Brace", e.g.:
>
> function foo()
> {
oops. looks like it's back to CODING_STANDARDS school for me ;)
more soon.
bo.
--
Bo Daley
Tilda Communications
http://www.tilda.com.au
bo at tilda.com.au
More information about the whups
mailing list