[Syncevolution-issues] [Bug 5049] sqlite: field list sync source
http://bugzilla.moblin.org/show_bug.cgi?id=5049
--- Comment #9 from Chen Congwu <congwu.chen(a)intel.com> 2009-09-28 21:39:56 ---
(In reply to comment #8)
(In reply to comment #7)
> Pushed to sqlite branch.
Comments:
* SyncSource::throwError() inserts "<source name>:" as prefix
automatically, code like this thus doubles the prefix;
getName() should not be used:
throwError(string(getName())+ "contact not found: " + uid)
* SQLiteUtil::prepareSQL: should use StringPrintf (from SyncEvolutionUtil.)
instead of vnsprintf
* --enable-sqlite: please keep the backend disabled by default.
This is something that developers should explicitly ask for.
Updated. SyncEvolutionUtil is really a good place to query before writing
something my own, :D
--
Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching someone on the CC list of the bug.