[dev] ansel status update

Ben Chavet ben at chavet.net
Thu Jan 8 11:06:24 PST 2004


> This looks quite good! A few comments:

Thanks!  I was afraid I was heading for another dead end

> - I hope there'll be some comments before it's committed? :)

Of course! :)

> - you mispelled flip. ;)

Oops!

> - It seems that some of the methods like renameImage, getImageParents, etc.
> could be bumped down into CategoryObject_Image, letting you call them just
> getParents, rename, etc. (and also to have just rename instead of
> renameGallery in Ansel_Gallery). Just a slightly more OO/cleaner API.

I had been debating that as well.  I figured I'd be able to iron those details
out when I actually had to start using the API.

One reason I went with the current method is to help with memory usage.  No
point in having multiple copies of a function when one can do the job.  Of
course, there aren't really going to be that many images in memory at one time
anyway...

I'll play with it & see what seems to work best.

--Ben

----------------------------------------------------------------
This message was sent using IMP, the Internet Messaging Program.


More information about the dev mailing list