search

yahoo / squidb

SquiDB is a SQLite database library for Android and iOS
https://github.com/yahoo/squidb/wiki
Apache License 2.0
1.31k stars 132 forks source link

Fix clone() #252

Closed sbosley closed 7 years ago

sbosley commented 7 years ago

Fixes bad handling of transitory values in AbstractModel.clone()

Closes #251

jdkoren commented 7 years ago

LGTM

jdkoren commented 7 years ago

Changes LGTM

sbosley commented 7 years ago

Update: in auditing the transitory code I noticed they would behave badly when using a null key or more importantly null values. Added fixes for that as well.

jdkoren commented 7 years ago

Changes still LGTM