Page MenuHomePhabricator

Use cstdint in generated test headers
ClosedPublic

Authored by deadalnix on Jul 7 2019, 13:09.

Details

Summary

It is more in line with the style we are adopting.

Test Plan
make clean
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cstdintcodegen
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6702
Build 11451: Bitcoin ABC Buildbot (legacy)
Build 11450: arc lint + arc unit

Event Timeline

markblundeberg added inline comments.
src/test/CMakeLists.txt
132

I can't see what's changing here, also... unrelated?

This revision now requires changes to proceed.Jul 8 2019, 01:36
deadalnix added inline comments.
src/test/CMakeLists.txt
132

It's a whitespace change. Someone misconfigured their text editor.

markblundeberg added inline comments.
src/test/CMakeLists.txt
132

Oh yeah I can see it now (by clicking 'view left file' and 'view right file'). Annoyingly, phab renders a tab as two spaces so they look identical here, hah.

This revision is now accepted and ready to land.Jul 8 2019, 13:18
This revision was automatically updated to reflect the committed changes.