/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-07-29 21:00:36 UTC
  • Revision ID: edam@waxworlds.org-20100729210036-7me19mqc1iqo34pm
- failed to update TODO in last commit

Show diffs side-by-side

added added

removed removed

 
1
 
 
2
IMMEDIATE ISSUES
 
3
 
 
4
- rename _bind_index and _column_index to _next_*
 
5
 
 
6
- turn on extended errcodes in open() and handle them in sqlite_error
 
7
 
 
8
- query::prepare() isn't being called during construction (form
 
9
        basic_statement's constructor)
 
10
 
1
11
- add columns() to row that returns a boost::tuple of various types so multple
2
12
        columns can be fetched at once (look in to using BOOST_PP_ITERATE macro)
3
13
 
 
14
- use sqlite3_db_mutex() to provide extended error information during
 
15
        sqlite_error construction. The general procedure would be to lock the db
 
16
        mutex, perform some sqlite3 command, check the error code, throw an
 
17
        sqlite_error (whilst obtaining extended error info) and then unlock the db
 
18
        mutex. Two options:
 
19
        - a macro would be simple
 
20
        - a templated safe-calling object (passing the comman's arg types as
 
21
                template params) may be overkill
 
22
 
 
23
 
 
24
LONGER TERM IDEAS
 
25
 
4
26
- make basic_statement and database keep a shared pointer to the database handle
5
27
        so the classes can be made copyable. The wrappers around the handle
6
28
        (implemented in sqlite::detail) can clean them up after use. This will also
10
32
                basic_statements, querys and commands copyable. Could weak_ptrs to these
11
33
                also be used in the database's list active querys?
12
34
 
13
 
- fix to force the finalisation of queries in progress for transactions causes
14
 
        errors; queries are now finalised twice, the second from basic_statement's
15
 
        dtor, which causes a segfault. We could:
16
 
        - keep a list of force-finalised sqlite3_stmt pointers in the database which
17
 
                we use to check queries against before finalising them to make sure we
18
 
                don't finalise them a second time
19
 
                - an efficient implementation, but not very OO
20
 
        - keep a map of active queries in the database (using the sqlite3_stmt
21
 
                pointer as the key), so that we can obtain the query and tell it to
22
 
                finalise its self
23
 
                - this seems like a messy and complicated implementation
24
 
 
25
 
- turn on extended errcodes in open() and handle them in sqlite_error
26
 
 
27
 
- use sqlite3_db_mutex() to provide extended error information during
28
 
        sqlite_error construction. The genreeal procedure would be to lock the db
29
 
        mutex, perform some sqlite3 command, check the error code, throw an
30
 
        sqlite_error (whilst obtaining extended error info) and then unlock the db
31
 
        mutex. Two options:
32
 
        - a macro would be simple
33
 
        - a templated safe-calling object (passing the comman's arg types as
34
 
                template params) may be overkill
35
 
 
36
35
- expand sqlite_error - perhaps use boost::system_error (see
37
36
        boost/asio/error.hpp for an example of extending system_error)
38
37
 
45
44
                        this matter? they can't access database._handle anyway!
46
45
                potential incompatibility when linking to libraries that also link
47
46
                        against sqlite
48
 
 
49
 
- query::prepare() isn't being called during construction (form
50
 
        basic_statement's constructor)