Removing static definition of namespace scoped method
Summary:
# Summary
- Removing `static` from the function definition as it causes build failures with unused-function errors: https://www.internalfb.com/intern/sandcastle/job/31525198029795004/insights
Reasoning:
```
When you declare (and define) a function as static in the header file, each translation unit that includes that header file gets its own internal copy of the function
```
More details in https://stackoverflow.com/questions/2845748/function-defined-but-not-used-warning-in-c
Test Plan: tests run in signal
Reviewed By: ezyang, seemethere
Differential Revision: D35727740
fbshipit-source-id: 9997a210a3e68e226ae6f8d51e76bd3431bb6754
(cherry picked from commit d52f99fb78acc2694636f92f9052412166f1adda)