Use PRIu64 to printf-format an uint64_t value

facebook / mysql-5.6

Facebook's branch of the Oracle MySQL database. This includes MyRocks.

http://myrocks.io

Other

2.46k stars 711 forks source link

Use PRIu64 to printf-format an uint64_t value #1439

Closed laurynas-biveinis closed 3 months ago

laurynas-biveinis commented 3 months ago

This fixes, on macOS:

storage/rocksdb/rdb_cmd_srv_helper.cc:115:52: error: format specifies type 'unsigned long' but the argument has type 'uint64_t' (aka 'unsigned long long') [-Werror,-Wformat]
                    "expect one row, but got %lu", row_count);
                                             ~~~   ^~~~~~~~~
                                             %llu

At the same time add some includes for the used declarations.

Squash with 7451f04c870763ad2f5ef0bc17e18ff96865e739

facebook-github-bot commented 3 months ago

@luqun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot commented 3 months ago

This pull request has been merged in facebook/mysql-5.6@277f5ccb43bd590c6e54db0506a677d92c6da7bd.