/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-27 15:12:55 UTC
  • Revision ID: edam@waxworlds.org-20100727151255-goaqgdz4kj13q7gz
- update TODO
- added some missing includes for <string>
- changed usage of database::exec() to not require return code!
- prevented transaction_guard destructor from throwing an exception

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
- add columns() to row that returns a boost::tuple of various types so multple columns can be fetched at once
 
2
        - look in to using BOOST_PP_ITERATE macro
 
3
- make basic_statement and database keep a shared pointer to the database handle so the classes can be made copyable. Also:
 
4
    - the wrappers around the handle can clean them up after use
 
5
    - the actual wrappers wround the handles can be made in sqlite::detail
 
6
    - this will also make the implementation od rows (to get round the forced non-dependency of rows on querys) a little easier to swallow.
 
7
- move _null_t, _exec_t and _set_index_t to sqlite::detail. Only the named instantiations of these structs need be in the sqlite namespace.
 
8
- add immediate_transaction
 
9
- committing a transaction during a query (i.e., when sqlite3_step() has returned SQLITE_ROW) causes an error. To counter this:
 
10
        - calling query.reset() before the commit fixes the issue
 
11
    - need to check it's not fixed in latest sqlite
 
12
    - will need to keep a list of querys that need resetting in the database  :o(
 
13
- turn on extended errcodes in open() and handle them in sqlite_error
 
14
- use sqlite3_db_mutex() to provide extended error information during sqlite_error construction - see sqlite::query::step() for example
1
15
- expand sqlite_error - perhaps use boost::system_error (see boost/asio/error.hpp for an example of extending system_error)
2
 
- use sqlite3_db_mutex() to provide extended error information during sqlite_error construction - see sqlite::query::step() for example
3
16
- see if we can #include "sqlite.h" in to a namespace.
4
17
        Pros:
5
18
                we better encapsulate the library
7
20
        Cons:
8
21
                makes access to real sqlite stuff awkward to sqlite3cc users, but does this matter? they can't access database._handle anyway!
9
22
                potential incompatibility when linking to libraries that also link against sqlite
10
 
- add immediate_transaction
11
23
- fix step() inconsistency - query::step() returns a row, whereas basic_statement::step() and command::step() return an int return code
12
 
- fix row/query compilation dependency issue. break dependency of querys on rows
13
 
        - can't use "row"s in the iterator
14
 
        - could wrap rows, and use dereference operator to access
15
 
        - could use a base class, which wouldn't cause extra dereferences in use and shouldn't have much overhead (no vfpt)
16
 
        - look at the boost::iterator_facade interface - can we switch to "row *"s?
17
24
- query::prepare() isn't being called during construction (form
18
25
        basic_statement's constructor)