Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace size_t with std::size_t #3426

Merged
merged 3 commits into from
Oct 21, 2019

Conversation

kunaltyagi
Copy link
Member

@kunaltyagi kunaltyagi commented Oct 20, 2019

A sed replacement done in 3 steps

  1. replace ([ (<,/])size_t with \1std::size_t
  2. replace std::size_type with size_type
  3. replace std::std:: with std::

All steps are squashed to one commit. Opennurbs is excluded since it is compiled in C (? huh) (Compiler threw an error and being the courageous deep diver I am, I undid the changes to make the compiler happy)

Might be a header situation too. Maybe someone else can help me

EDIT: Opennurbs edited. 3 commits now: replace, format and headers

@SergioRAgostinho SergioRAgostinho merged commit 8eb4768 into PointCloudLibrary:master Oct 21, 2019
@kunaltyagi kunaltyagi deleted the std_size branch October 22, 2019 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants