Skip to content

Mesh_2: Fix write_vtu in ASCII#5679

Merged
lrineau merged 2 commits intoCGAL:5.1.x-branchfrom
lrineau:Mesh_2-fix_write_VTU_in_ASCII-GF
May 18, 2021
Merged

Mesh_2: Fix write_vtu in ASCII#5679
lrineau merged 2 commits intoCGAL:5.1.x-branchfrom
lrineau:Mesh_2-fix_write_VTU_in_ASCII-GF

Conversation

@lrineau
Copy link
Copy Markdown
Member

@lrineau lrineau commented May 5, 2021

Summary of Changes

write_vtu, from PR #3297 (CGAL-4.14) was buggy with ASCII_MODE, because of spurious " in the output file.

I found out while trying to debug the experimental NURBS mesher.

Release Management

  • Affected package(s): Mesh_2
  • License and copyright ownership: maintenance by GF

@sloriot
Copy link
Copy Markdown
Member

sloriot commented May 5, 2021

This will conflict with #5677

@sloriot
Copy link
Copy Markdown
Member

sloriot commented May 5, 2021

I moved the file + renamed the function using _VTU

@maxGimeno
Copy link
Copy Markdown
Contributor

Is this bug still in master ? This PR strongly conflicts with #5677 , as the whole file has been made deprecated

@lrineau
Copy link
Copy Markdown
Member Author

lrineau commented May 5, 2021

I moved the file + renamed the function using _VTU

The patch probably applies to the renamed file in #5677. You can apply it directly in the pull-request.

@lrineau
Copy link
Copy Markdown
Member Author

lrineau commented May 5, 2021

Is this bug still in master ? This PR strongly conflicts with #5677 , as the whole file has been made deprecated

Yes, the bug is still in master, because that file was not substantially modified (except for cosmetic changes, like spaces-at-EOLs) since its introduction in CGAL-4.14.

I will modify #5677 and merged this #5679 in it directly.

lrineau added a commit to sloriot/cgal that referenced this pull request May 5, 2021
@lrineau lrineau mentioned this pull request May 5, 2021
lrineau added a commit to lrineau/cgal that referenced this pull request May 12, 2021
@maxGimeno
Copy link
Copy Markdown
Contributor

@lrineau lrineau added rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' rm only: ready for release branch For the release team only: that indicates that a PR is about to be merged in a release branch labels May 18, 2021
@lrineau lrineau changed the base branch from master to 5.1.x-branch May 18, 2021 14:50
lrineau added a commit that referenced this pull request May 18, 2021
lrineau added a commit that referenced this pull request May 18, 2021
@lrineau lrineau merged commit 2b6031a into CGAL:5.1.x-branch May 18, 2021
@lrineau lrineau removed rm only: ready for release branch For the release team only: that indicates that a PR is about to be merged in a release branch rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' labels May 18, 2021
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.

3 participants