-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Sven Neumann wrote: | Hi, | | Daniel Rogers <daniel@xxxxxxxxxxxxxxxxx> writes: | | |>1. Eliminating TileManager and GimpDrawable and replacing it with |>GeglBufferedImage. (GeglBufferedImage will be the name of the concrete |>GeglImage subclass that just stores its Tiles in a GeglBuffer (as |>opposed to fetching them from an op)). GimpDrawable is the thing |>closest in concept to a GeglImage, atm. | | | You can hardly replace GimpDrawable with a GEGL type since it is | itself derived from a GIMP type and it wouldn't make sense to change | that. Please forget about introducing GEGL types into the GIMP object | hierarchy. This simply won't work. All you can do is to make | GimpDrawable keep a pointer to a GeglImage just like it has a pointer | to a TileManager right now. Right right, I meant do a reasonable thing, not an unreasonable thing. I have been looking for places where concepts in the gimp and gegl overlap and suggesting that gegl do the work there instead. Delegation is a perfectly reasonable solution. | |>3. Modifing GimpImage to GeglImage. Currently, GimpImage is more |>akin to a stack of GeglImages. This change shouldn't be too hard |>(at first), from what I can tell. Mostly, the stuff that changes |>will be the things that use a GimpImage to get the TileManagers and |>GimpDrawables, and those will need to change to to acomodate (1) |>anyway. Anywhoo, the GimpImage, at this point, will do same thing, |>just keep a few pieces of data in a difference place. | | | The same comment applies here. A GimpImage is a lot more than stacks | of layers and channels. It has an awful lot of GIMP specific | properties that you certainly don't want to add to GEGL. You shouldn't | attempt to replace it in anyway. Same as above: delegation is a perfectly reasonable solution here. Do you think my roadmap is at least reasonable, except for these details? - -- Dan -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.3 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFAArtSad4P1+ZAZk0RAsSUAJ0f2TSZmPqrmYDqcLIN7aMDviHHjQCfS9T5 FWbXcj0u9VBjeBn7BLOn1so= =rt9E -----END PGP SIGNATURE-----