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

Fix const iterator issue1437 #1475

Merged
merged 6 commits into from
Apr 10, 2020

Conversation

kerautret
Copy link
Member

@kerautret kerautret commented Apr 9, 2020

PR Description

See issue #1437

Checklist

  • Unit-test of your feature with Catch.
  • Doxygen documentation of the code completed (classes, methods, types, members...)
  • Documentation module page added or updated.
  • New entry in the ChangeLog.md added.
  • No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).
  • All continuous integration tests pass (Travis & appveyor)

@kerautret kerautret changed the title Fix const iterator issue1437 WIP Fix const iterator issue1437 Apr 9, 2020
@kerautret kerautret changed the title WIP Fix const iterator issue1437 Fix const iterator issue1437 Apr 9, 2020
ChangeLog.md Outdated
@@ -135,7 +135,10 @@
(David Coeurjolly, [#1446](https:/DGtal-team/DGtal/pull/1446))
- Removes the check for CPP11 when building WITH_ITK
(Pablo Hernandez-Cerdan, [#1453](https:/DGtal-team/DGtal/pull/1453))

- Fix apple clang compilation issue with a workaround to the
ConstIteratorAdapter that do not satisfy the _is_forward concept of the STL:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
ConstIteratorAdapter that do not satisfy the _is_forward concept of the STL:
ConstIteratorAdapter class that does not satisfy the _is_forward concept of the STL:

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, I miss the commit suggestion since my emacs was open on it but that looks nice!

@dcoeurjo
Copy link
Member

thx for the PR

@dcoeurjo
Copy link
Member

All good ;)

@dcoeurjo dcoeurjo merged commit 543c456 into DGtal-team:master Apr 10, 2020
dcoeurjo added a commit that referenced this pull request Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants