Page MenuHomePhabricator

Merge #9544: [trivial] Add end of namespace comments. Improve consistency.
ClosedPublic

Authored by deadalnix on Jul 9 2019, 19:49.

Details

Summary

5a9b508 [trivial] Add end of namespace comments (practicalswift)

Tree-SHA512: 92b0fcae4d1d3f4da9e97569ae84ef2d6e09625a5815cd0e5f0eb6dd2ecba9852fa85c184c5ae9de5117050330ce995e9867b451fa8cd5512169025990541a2b

Backport of Core PR9544
https://github.com/bitcoin/bitcoin/pull/9544/files

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nakihito created this revision.Jul 9 2019, 19:49
Owners added a reviewer: Restricted Owners Package.Jul 9 2019, 19:49
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 9 2019, 19:49
nakihito planned changes to this revision.Jul 9 2019, 19:49
nakihito requested review of this revision.Jul 10 2019, 22:06

base58.cpp change wes made in D624 and then the code was moved to a different file in D3577.
The majority of other changes were made in D873.

deadalnix requested changes to this revision.Jul 10 2019, 23:56

There is a tool that can do this automatically and exaustively. Use it.

This revision now requires changes to proceed.Jul 10 2019, 23:56
deadalnix commandeered this revision.Jul 14 2019, 17:49
deadalnix edited reviewers, added: nakihito; removed: deadalnix.
deadalnix updated this revision to Diff 10288.Jul 14 2019, 17:53

Update cland-format to enforce the new style

nakihito accepted this revision.Jul 15 2019, 22:39
jasonbcox accepted this revision.Jul 16 2019, 03:01
This revision is now accepted and ready to land.Jul 16 2019, 03:01