Summary:
Declaring a return value as `const` doesn't really make sense, and the
compiler simply ignores the keyword.
This removes the `const` keyword so that the code compiles cleanly when built
with `-Wignored-qualifiers`
Reviewed By: quark-zju
Differential Revision: D16635863
fbshipit-source-id: 5c1fd3689c7e8f5a68ba9211d6581b3a59f9cf28
Summary:
They're actively fighting against the clang-format config
and don't have an auto-fix.
Reviewed By: quark-zju
Differential Revision: D8283622
fbshipit-source-id: 2de45f50e6370a5ed14915c6ff23dc843ff14e8a
Summary:
1. Variable Length Arrays are not supported by MSVC, but since this is a C++ code, we can just use heap allocation
2. Replacing `inet` with portability version
Depends on D7196403
Reviewed By: quark-zju
Differential Revision: D7196605
fbshipit-source-id: a0d88b6e06f255ef648c0b35a99b42ba3bee538a
Summary:
This moves the cdatapack code to the new lib/ directory and adds it to the main
setup.py.
Test Plan: hg purge --all && make local && cd tests && ./run-tests.py -S -j 48
Reviewers: #mercurial
Differential Revision: https://phabricator.intern.facebook.com/D6677491
