azure-sdk-for-cpp icon indicating copy to clipboard operation
azure-sdk-for-cpp copied to clipboard

Pick one `std::(u)intX_t` vs `(u)intX_t`, `std::size_t` vs `size_t`, ptrdiff_t, etc, and update the sources to use that one form

Open antkmsft opened this issue 2 years ago • 1 comments

Additionally, scan if we use NULL (in non-3rd party source) and update to nullptr.

antkmsft avatar Nov 09 '23 22:11 antkmsft

For c++ code, I believe that the only "correct" version is the std::xxx version.

For correctness, the standard declares that the std::xxx type exactly matches the xxx type, so there is no generated code correctness issue associated with either selection.

However, to complicate this, we should absolutely allow putting a using namespace std; directive at the top of source files.

LarryOsterman avatar Nov 09 '23 22:11 LarryOsterman

Hi @antkmsft, we deeply appreciate your input into this project. Regrettably, this issue has remained unresolved for over 2 years and inactive for 30 days, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.

github-actions[bot] avatar Nov 10 '25 18:11 github-actions[bot]