quasselgrep
quasselgrep copied to clipboard
Missing timestamps #19
This PR fixes issue #19:
- on sqlite3 >= 31, timestamps now display properly
- on sqlite3 >= 31, filtering based on timestamps (-t option) now works
For sqlite3 databases, we read schemaversion on connect and store it into the options object. That way, we can deduce the timestamp unit (milliseconds or seconds) from other files. This approach was suggested by @digitalcircuit in #19 [1].
I've just started using quasselgrep, so perhaps I missed something. Let me know if that's the case. Also, I cannot test with databases < 31, but old behavior should be left untouched in that case.
[1] https://github.com/fish-face/quasselgrep/issues/19#issuecomment-482753420
Hello, anything preventing the merge :)?