lp:~stewart/drizzle/cursor-warn-unused-result

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:drizzle/7.0 at revision 2067

Drizzle Developers: Pending requested 2011-01-07

Diff: 270 lines (+91/-27)

8 files modified

drizzled/cursor.cc (+6/-3)
drizzled/cursor.h (+5/-5)
drizzled/item/subselect.cc (+19/-2)
drizzled/records.cc (+11/-5)
drizzled/records.h (+4/-4)
drizzled/sql_delete.cc (+7/-1)
drizzled/sql_select.cc (+37/-6)
drizzled/sql_update.cc (+2/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #697973: drizzled::Cursor::read_first_row() ignores return value of startIndexScan	Medium	Fix Released
Bug #697982: drizzled::subselect_uniquesubquery_engine::exec() and drizzled::subselect_indexsubquery_engine::exec() ignores startindexScan result	Medium	Fix Released
Bug #697984: drizzled::ReadRecord::init_read_record_idx ignores startIndexScan result	Medium	Fix Released
Bug #698046: startIndexScan result largely ignored in sql_select.cc	Medium	Fix Released

Link a bug report

Related blueprints

2060. By Stewart Smith on 2011-01-07

add warn_unused_result to Cursor::index_next

2059. By Stewart Smith on 2011-01-06

add attribute(warn_unused_result) to Cursor::startIndexScan

2058. By Stewart Smith on 2011-01-06

check return value of Cursor::startIndexScan in sql_select

2057. By Stewart Smith on 2011-01-06

init_read_record_idx return result should be checked now that it checks startIndexScan result. Fix case in sql_update

2056. By Stewart Smith on 2011-01-06

init_read_record_idx return result should be checked now that it checks startIndexScan result.

2055. By Stewart Smith on 2011-01-06

the return value of Cursor::startIndexScan was not being checked in 2 places in item/subselect.cc.

2054. By Stewart Smith on 2011-01-06

check return value of Cursor::startIndexScan in read_first_row

2053. By Stewart Smith on 2011-01-06

add warn_unused_result for updateRecord and deleteRecord

2052. By Stewart Smith on 2011-01-06

add warn_unused_result to Cursor::insertRecord

2051. By Stewart Smith on 2011-01-05

remove the saving of previous_error in doStartTableScan in HailDB. Now that we've fixed the upper layer to handle errors returned from doStarttableScan correctly.