[Shotwell] Source code reorganization

Bruno Girin brunogirin at gmail.com
Wed Jan 12 23:40:39 UTC 2011


Hi Jim,

That's great news and will make it a lot easier to navigate the code!
When I have the time, hopefully in the next few weeks, I'll re-organise
the AlienDatabase/F-Spot import code into units. However, this brings a
question: are you happy for me to make an exception to the "one-deep"
nesting and nest an fspot unit inside an alien_database one?

Cheers,

Bruno

On Tue, 2011-01-11 at 18:53 -0800, Jim Nelson wrote:
> One item that's been on my mind for a while and was discussed here
> previously (
> http://lists.yorba.org/pipermail/shotwell/2010-September/001088.html) is
> reorganizing Shotwell's source code.  With whitespace and comments, Shotwell
> is now over 70,000 lines of code.  The old model -- current before about an
> hour ago -- was a flat list of files in a single directory with minimal use
> of namespaces.  I've checked in to trunk a new model.
> 
> Rather than go on about it here, anyone interested should read a page I've
> written about it on our architectural overview:
> http://trac.yorba.org/wiki/ShotwellArchSourceOrganization
> 
> If you're currently working on a patch, you'll want to update your local
> copy and merge with the new code before submitting.
> 
> Further work will follow to move more of the code into separate units and
> further clean up the organization.  This is a first step toward a more
> manageable code base.  Hopefully from here on it's smaller, more incremental
> changes.
> 
> Cheers,
> 
> -- Jim
> _______________________________________________
> Shotwell mailing list
> Shotwell at lists.yorba.org
> http://lists.yorba.org/cgi-bin/mailman/listinfo/shotwell





More information about the Shotwell-list mailing list