/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 20:56:19 UTC
  • Revision ID: edam@waxworlds.org-20100729205619-a9yo4uzu647nvnsj
- renamed database to connection to better identify what it is (would database_connection be better though?)
- moved basic_statement and basic_transaction to sqlite::detail
- made sqlite::threadsafe() return the threading mode int, not a bool
- renamed row::index() to row_number() so it isn't confused with column index
- added typedef for deferred_transaction
- added early rollback method to transaction_guard
- allowed transaction_guard::~transaction_guard() to throw exceptions, since when it needs to, we're really screwed anyway
- bugfix: query::reset() didn't reset the internal row counter
- added query::num_results()
- added docs/design-notes
- reimplemented query::iterator so that increment() (which performs a step() on the query) now caches the returned row to be returned during dereference() (previously it stashed details and returned new row!?)
- bugfix: resetting active queries during rollbacks would hang!

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_*
 
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?
5
4
 
6
5
- turn on extended errcodes in open() and handle them in sqlite_error
7
6
 
8
 
- query::prepare() isn't being called during construction (form
9
 
        basic_statement's constructor)
 
7
- make basic_statement and database keep a shared pointer to the database handle
 
8
        so the classes can be made copyable. The wrappers around the handle
 
9
        (implemented in sqlite::detail) can clean them up after use. This will also
 
10
        make the implementation of rows (to get round the forced non-dependency of
 
11
        rows on querys) a little easier to swallow.
 
12
        - A similar wrapper should be created for statement handles, making
 
13
                basic_statements, querys and commands copyable. Could weak_ptrs to these
 
14
                also be used in the database's list active querys?
10
15
 
11
16
- add columns() to row that returns a boost::tuple of various types so multple
12
17
        columns can be fetched at once (look in to using BOOST_PP_ITERATE macro)
20
25
        - a templated safe-calling object (passing the comman's arg types as
21
26
                template params) may be overkill
22
27
 
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
28
- expand sqlite_error - perhaps use boost::system_error (see
36
29
        boost/asio/error.hpp for an example of extending system_error)
37
30
 
44
37
                        this matter? they can't access database._handle anyway!
45
38
                potential incompatibility when linking to libraries that also link
46
39
                        against sqlite
 
40
 
 
41
- query::prepare() isn't being called during construction (form
 
42
        basic_statement's constructor)