/sqlite3cc

To get this branch, use:
bzr branch http://bzr.ed.am/sqlite3cc

« back to all changes in this revision

Viewing changes to TODO

  • Committer: edam
  • Date: 2010-02-08 22:08:57 UTC
  • Revision ID: edam@waxworlds.org-20100208220857-5up6fi4y6jh5bz4k
- removed "OK" from test-main when test is successful
- aranged for test/test-main to buld properly and be called from "make check"
- added include/Makefile.am to arrange for installation of library headers
- added project website to configure.ac
- added library interface configuration variable to configure.ac

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
- fix to force the finalisation of queries in progress for transactions causes
2
 
        errors; queries are now finalised twice, the second from basic_statement's
3
 
        dtor, which causes a segfault. We could:
4
 
        - keep a set of force-finalised sqlite3_stmt pointers in the database which
5
 
                we use to check queries against before finalising them to make sure we
6
 
                don't finalise them a second time
7
 
                - an efficient implementation, but not very OO
8
 
        - keep a map of active queries in the database (using the sqlite3_stmt
9
 
                pointer as the key), so that we can obtain the query and tell it to
10
 
                finalise its self
11
 
                - this seems like a messy and complicated implementation
12
 
                - would be improved by having a wrapper around the query handle
13
 
 
14
 
- turn on extended errcodes in open() and handle them in sqlite_error
15
 
 
16
 
- make basic_statement and database keep a shared pointer to the database handle
17
 
        so the classes can be made copyable. The wrappers around the handle
18
 
        (implemented in sqlite::detail) can clean them up after use. This will also
19
 
        make the implementation of rows (to get round the forced non-dependency of
20
 
        rows on querys) a little easier to swallow.
21
 
        - A similar wrapper should be created for statement handles, making
22
 
                basic_statements, querys and commands copyable. Could weak_ptrs to these
23
 
                also be used in the database's list active querys?
24
 
 
25
 
- add columns() to row that returns a boost::tuple of various types so multple
26
 
        columns can be fetched at once (look in to using BOOST_PP_ITERATE macro)
27
 
 
28
 
- use sqlite3_db_mutex() to provide extended error information during
29
 
        sqlite_error construction. The genreeal procedure would be to lock the db
30
 
        mutex, perform some sqlite3 command, check the error code, throw an
31
 
        sqlite_error (whilst obtaining extended error info) and then unlock the db
32
 
        mutex. Two options:
33
 
        - a macro would be simple
34
 
        - a templated safe-calling object (passing the comman's arg types as
35
 
                template params) may be overkill
36
 
 
37
 
- expand sqlite_error - perhaps use boost::system_error (see
38
 
        boost/asio/error.hpp for an example of extending system_error)
39
 
 
40
 
- see if we can #include "sqlite.h" in to a namespace.
41
 
        Pros:
42
 
                we better encapsulate the library
43
 
                we can reuse "sqlite3" as a namespace
44
 
        Cons:
45
 
                makes access to real sqlite stuff awkward to sqlite3cc users, but does
46
 
                        this matter? they can't access database._handle anyway!
47
 
                potential incompatibility when linking to libraries that also link
48
 
                        against sqlite
49
 
 
50
 
- query::prepare() isn't being called during construction (form
51
 
        basic_statement's constructor)
 
1
- turn transaction in to basic_transaction and create transaction, exclusive_transaction and perhaps recursive_transaction?
 
2
- expand sqlite_error - perhaps use boost::system_error (see boost/asio/error.hpp for an example of extending system_error)
 
3
- see if we can #include "sqlite.h" in to a namespace. Cons: no easy access to real sqlite stuff. Pros: can't access database::_Handle anyway, so Cons don't matter and we better encapsulate the library
 
4
- fix "make check"