Skip to content

Forcing static type in VoxelComplex.ih#1823

Merged
dcoeurjo merged 2 commits intomainfrom
compilerIssuefabienbaldaci
Dec 6, 2025
Merged

Forcing static type in VoxelComplex.ih#1823
dcoeurjo merged 2 commits intomainfrom
compilerIssuefabienbaldaci

Conversation

@dcoeurjo
Copy link
Copy Markdown
Member

@dcoeurjo dcoeurjo commented Dec 5, 2025

PR Description

See #1820

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 mode.
  • All continuous integration tests pass (Github Actions)

@JacquesOlivierLachaud
Copy link
Copy Markdown
Member

I have a cmake error with the current DGtal branches (both main and this one).

-- Configuring done (1.8s)
CMake Error in CMakeLists.txt:
  export called with target "DGtal" which requires target "Eigen3_Eigen" that
  is not in any export set.
-- Generating done (0.2s)
CMake Generate step failed.  Build files cannot be regenerated correctly.
make: *** [cmake_check_build_system] Error 1

This is probably linked to a former commit.

@dcoeurjo
Copy link
Copy Markdown
Member Author

dcoeurjo commented Dec 5, 2025

Yes it seems unrelated to this PR. Could you please fill up an issue with details on your OS/cmake version/ compiler... ?

It is fine on my computer and GitHub actions bots

@dcoeurjo
Copy link
Copy Markdown
Member Author

dcoeurjo commented Dec 6, 2025

I will merge this one.

@dcoeurjo dcoeurjo merged commit 0460ff1 into main Dec 6, 2025
6 checks passed
@dcoeurjo dcoeurjo deleted the compilerIssuefabienbaldaci branch December 6, 2025 15:05
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.

Compilation error from ambiguous overloading of operator <<

2 participants