/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 06:28:53 UTC
  • Revision ID: edam@waxworlds.org-20100729062853-4i2fec52m86mh724
- made basic_statement::step() protected, for use by query and command only
- moved basic_statement::operator<<() to command and query instead; one needs to accept sqlite::exec, the other doesn't
- added tests for query::operator<<()
- added code to invlaidate in-progress queries during any transaction rollbacks (currently segfaults in basic_statement::finalize())
- added new sqlite_error constructor that obtains a full error message
- implemented database::database_mutex_guard class
- swapped command's step mutex in favour of the database mutex

Show diffs side-by-side

added added

removed removed

Lines of Context:
TODO
1
 
- check that the fix for in-progress queries during rollback is threadsafe. In
2
 
        particular, we shouldn't be resetting queries from another thread! Does
3
 
        sqlite3_next_stmt() only return statements from this thread?
4
 
 
5
 
- turn on extended errcodes in open() and handle them in sqlite_error
 
1
- add columns() to row that returns a boost::tuple of various types so multple
 
2
        columns can be fetched at once (look in to using BOOST_PP_ITERATE macro)
6
3
 
7
4
- make basic_statement and database keep a shared pointer to the database handle
8
5
        so the classes can be made copyable. The wrappers around the handle
13
10
                basic_statements, querys and commands copyable. Could weak_ptrs to these
14
11
                also be used in the database's list active querys?
15
12
 
16
 
- add columns() to row that returns a boost::tuple of various types so multple
17
 
        columns can be fetched at once (look in to using BOOST_PP_ITERATE macro)
 
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
18
26
 
19
27
- use sqlite3_db_mutex() to provide extended error information during
20
 
        sqlite_error construction. The general procedure would be to lock the db
 
28
        sqlite_error construction. The genreeal procedure would be to lock the db
21
29
        mutex, perform some sqlite3 command, check the error code, throw an
22
30
        sqlite_error (whilst obtaining extended error info) and then unlock the db
23
31
        mutex. Two options: