On 04/06/11 04:43 -0700, reichlich wrote: > Hi all, > > i would like to show you the status of my POS-project. By now simple > synching is possible as shown in a short screencast on youtube: > http://www.youtube.com/watch?v=iAGP5fpdOAM > > It would be nice if some of you look at the code and give me some > feedback. > A mercurial repos is available under http://hg.holtzberg.de/synchronize
Interesting. I think as the design is complex a small doc entry about it will be great. Also as it is not trivial algorithm you should write tests (like for recursion etc.) I see you use executemany instead of the standard way. I suppose it is for performance but I'm not sure it is a good design choice because you duplicate a lot of stuffs (like value formating), you lost the abilities of writing Function fields. Also I think it will be good to be able to commit particial synchronisation. Regards, -- Cédric Krier B2CK SPRL Rue de Rotterdam, 4 4000 Liège Belgium Tel: +32 472 54 46 59 Email/Jabber: [email protected] Website: http://www.b2ck.com/
pgp9KYjvmtaJK.pgp
Description: PGP signature
