/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 09:16:26 UTC
  • Revision ID: edam@waxworlds.org-20100729091626-h8fmg0r74eyfo5ae
- fixed error caused by finialising in-progress queries during rollback that were later finaliased by RAII.

Show diffs side-by-side

added added

removed removed

Lines of Context:
TODO
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)
3
 
 
4
 
- make basic_statement and database keep a shared pointer to the database handle
5
 
        so the classes can be made copyable. The wrappers around the handle
6
 
        (implemented in sqlite::detail) can clean them up after use. This will also
7
 
        make the implementation of rows (to get round the forced non-dependency of
8
 
        rows on querys) a little easier to swallow.
9
 
        - A similar wrapper should be created for statement handles, making
10
 
                basic_statements, querys and commands copyable. Could weak_ptrs to these
11
 
                also be used in the database's list active querys?
12
 
 
13
1
- fix to force the finalisation of queries in progress for transactions causes
14
2
        errors; queries are now finalised twice, the second from basic_statement's
15
3
        dtor, which causes a segfault. We could:
16
 
        - keep a list of force-finalised sqlite3_stmt pointers in the database which
 
4
        - keep a set of force-finalised sqlite3_stmt pointers in the database which
17
5
                we use to check queries against before finalising them to make sure we
18
6
                don't finalise them a second time
19
7
                - an efficient implementation, but not very OO
21
9
                pointer as the key), so that we can obtain the query and tell it to
22
10
                finalise its self
23
11
                - this seems like a messy and complicated implementation
 
12
                - would be improved by having a wrapper around the query handle
24
13
 
25
14
- turn on extended errcodes in open() and handle them in sqlite_error
26
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
 
27
28
- use sqlite3_db_mutex() to provide extended error information during
28
29
        sqlite_error construction. The genreeal procedure would be to lock the db
29
30
        mutex, perform some sqlite3 command, check the error code, throw an