sql: Remove basictypes.h includes.

sql: Remove basictypes.h includes.

* Use the standard integer types from stdint.h instead.
* Use macros.h for the DISALLOW_COPY_AND_ASSIGN macro.

BUG=138542
TEST=sql_unittests
R=shess@chromium.org

Committed: https://crrev.com/720d4f346cee3fa9ea9c58b845550e7066a80ec2
Cr-Commit-Position: refs/heads/master@{#329256}

[Scott Hess - ex-Googler, 2015-05-11 20:34:23]

Please s/int64/int64_t/ in the unit tests and test/test_helpers.cc in this CL.

https://codereview.chromium.org/1133053004/diff/1/sql/statement.cc
File sql/statement.cc (right):

https://codereview.chromium.org/1133053004/diff/1/sql/statement.cc#newcode303
sql/statement.cc:303: std::vector<unsigned char>* val) const {
If this is clang-format results, then that's fine.  If not, please revert this
piece, no need to inject irrelevant whitespace changes.

[tfarina, 2015-05-11 20:39:07]

Done. PTAL

https://codereview.chromium.org/1133053004/diff/1/sql/statement.cc
File sql/statement.cc (right):

https://codereview.chromium.org/1133053004/diff/1/sql/statement.cc#newcode303
sql/statement.cc:303: std::vector<unsigned char>* val) const {
On 2015/05/11 20:34:23, Scott Hess wrote:
> If this is clang-format results, then that's fine.
It was. Sorry, I should not have mixed this with this change, it happened while
I was scanning this file. Reverted.

[Scott Hess - ex-Googler, 2015-05-11 20:49:51]

thanks, lgtm.

[tfarina, 2015-05-11 22:22:54]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2015-05-11 22:23:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1133053004/20001

[commit-bot: I haz the power, 2015-05-11 22:31:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-11 22:32:28]

Patchset 2 (id:??) landed as
https://crrev.com/720d4f346cee3fa9ea9c58b845550e7066a80ec2
Cr-Commit-Position: refs/heads/master@{#329256}