On Tue, Aug 25, 2009 at 10:23:47PM +0800, Ryan Li wrote:
> And here's the patch for threaded backup. I use the Thread class for each
> device, and used it to handle ConfigFile and DeviceIface; and class
> DeviceBus is used to probe connected devices, which returns a Device
> instance.
> 
> I modified a lot of code and not sure whether you'll like it.

You're right.... that was a big patch, but it was a good patch too.
I liked the modular work you did in rearranging the code.  Nice.

I've applied the patch, with only a few minor tweaks (which you can see
in the git commits).

I think the GUI safeguards in general need more testing though.
For example, if a backup is going, you can use File | Quit with
no warning popping up.  Things like that probably need to be nailed
down a bit more, but otherwise, I like it a lot.

Thanks!
- Chris


------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Barry-devel mailing list
Barry-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/barry-devel

Reply via email to