/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-23 12:57:07 UTC
  • Revision ID: edam@waxworlds.org-20100723125707-qu9jk9vvg2uewx7t
- cleaned up test-main
- fixed comment type
- made sqlite::exec() throw instead of returning sqlite error code

Show diffs side-by-side

added added

removed removed

Lines of Context:
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
 
 
11
 
- add columns() to row that returns a boost::tuple of various types so multple
12
 
        columns can be fetched at once (look in to using BOOST_PP_ITERATE macro)
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
 
 
26
 
- make basic_statement and database keep a shared pointer to the database handle
27
 
        so the classes can be made copyable. The wrappers around the handle
28
 
        (implemented in sqlite::detail) can clean them up after use. This will also
29
 
        make the implementation of rows (to get round the forced non-dependency of
30
 
        rows on querys) a little easier to swallow.
31
 
        - A similar wrapper should be created for statement handles, making
32
 
                basic_statements, querys and commands copyable. Could weak_ptrs to these
33
 
                also be used in the database's list active querys?
34
 
 
35
 
- expand sqlite_error - perhaps use boost::system_error (see
36
 
        boost/asio/error.hpp for an example of extending system_error)
37
 
 
 
1
- 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
38
3
- see if we can #include "sqlite.h" in to a namespace.
39
4
        Pros:
40
5
                we better encapsulate the library
41
6
                we can reuse "sqlite3" as a namespace
42
7
        Cons:
43
 
                makes access to real sqlite stuff awkward to sqlite3cc users, but does
44
 
                        this matter? they can't access database._handle anyway!
45
 
                potential incompatibility when linking to libraries that also link
46
 
                        against sqlite
 
8
                makes access to real sqlite stuff awkward to sqlite3cc users, but does this matter? they can't access database._handle anyway!
 
9
                potential incompatibility when linking to libraries that also link against sqlite
 
10
- add immediate_transaction
 
11
- 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
- query::prepare() isn't being called during construction (form
 
18
        basic_statement's constructor)