// fprintf(stderr, "Starting\n");
- - if (db == NULL || target == NULL || local_gen_before_sync == NULL) {
+ if (db == NULL || target == NULL) {
// fprintf(stderr, "DB, target, or local are NULL\n");
You should probably still check that "local_gen == NULL" here, so that
we generate an INVALID_PARAMETER error if it is not supplied. Or are
you intentionally making it optional? If so, you should update the
docstring to indicate that.
The rest looks good.
merge: approve
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 8/15/2012 2:42 PM, Eric Casteleijn wrote: /code.launchpad .net/~thisfred/ u1db/fix- local-gen- before- sync/+merge/ 119705
> Eric Casteleijn has proposed merging
> lp:~thisfred/u1db/fix-local-gen-before-sync into lp:u1db.
>
> Requested reviews: Ubuntu One hackers (ubuntuone-hackers)
>
> For more details, see:
> https:/
>
> Fixed local_gen to be an OUT parameter only, and return it from
> sync()
>
// fprintf(stderr, "Starting\n"); before_ sync == NULL) {
- - if (db == NULL || target == NULL || local_gen_
+ if (db == NULL || target == NULL) {
// fprintf(stderr, "DB, target, or local are NULL\n");
You should probably still check that "local_gen == NULL" here, so that
we generate an INVALID_PARAMETER error if it is not supplied. Or are
you intentionally making it optional? If so, you should update the
docstring to indicate that.
The rest looks good.
merge: approve
-----BEGIN PGP SIGNATURE----- enigmail. mozdev. org/
rrJsACgkQJdeBCY SNAAPoygCgnAVKF l2EoKukAwlohz/ DRhsd AqJR4S5yfIIlgC9 tQDedtt
Version: GnuPG v1.4.12 (Cygwin)
Comment: Using GnuPG with Mozilla - http://
iEYEARECAAYFAlA
5bsAn3vlT/
=dSwQ
-----END PGP SIGNATURE-----