New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 908690 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

sql::Statement::BindInt should raise build error on implicit integer down-conversion

Project Member Reported by carlosk@chromium.org, Nov 27

Issue description

I recently stumbled into a but mostly by chance that involved the incorrect usage of sql::Statement::BindInt instead of BindInt64, that caused an implicit down-conversion from int64 to int32 and saved an incorrect value into the DB.

It would be great if these implicit conversions would in fact generate a build error (or at least a DCHECK) so that we could more easily identify these subtle issues.

 

Sign in to add a comment