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.

Sign up for GitHub

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

Jump to bottom

MVE: Image class should use 'int' type instead of 'std::size_t' #18

Closed
simonfuhrmann opened this issue Nov 28, 2012 · 1 comment
Closed

MVE: Image class should use 'int' type instead of 'std::size_t' #18

simonfuhrmann opened this issue Nov 28, 2012 · 1 comment
Labels
bug

Comments

@simonfuhrmann
Copy link
Owner

Reasons pro 'int':

  • Potentially dangerous to do arithmetic on std::size_t (underflow)
  • std::size_t is not a C++ base type thus requires STL includes
  • more convenient to use in arithmetics, shorter type name
@simonfuhrmann
Copy link
Owner Author

Refactorings are completed.

@supertobi supertobi mentioned this issue Nov 4, 2016
Closed
MasterShockwave pushed a commit to MasterShockwave/mve that referenced this issue Oct 20, 2019
@flanggut
Merge pull request simonfuhrmann#18 from bartoszek/patch-1
ee45c73 
MVE rename file string.h > strings.h
MasterShockwave pushed a commit to MasterShockwave/mve that referenced this issue Oct 20, 2019
@andre-schulz
Add featurerecon to build system
e0d5d8a 
Fixes issue simonfuhrmann#18; thanks to @nsn88 for reporting the issue.
MasterShockwave pushed a commit to MasterShockwave/mve that referenced this issue Oct 25, 2019
@andre-schulz
Add featurerecon to build system
aaf7b62 
Fixes issue simonfuhrmann#18; thanks to @nsn88 for reporting the issue.
MasterShockwave pushed a commit to MasterShockwave/mve that referenced this issue Aug 16, 2020
@andre-schulz
Add featurerecon to build system
e622099 
Fixes issue simonfuhrmann#18; thanks to @nsn88 for reporting the issue.
MasterShockwave pushed a commit to MasterShockwave/mve that referenced this issue Apr 6, 2021
@andre-schulz
Add featurerecon to build system
852efea 
Fixes issue simonfuhrmann#18; thanks to @nsn88 for reporting the issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@simonfuhrmann