View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002727 | OpenFOAM | Bug | public | 2017-10-18 15:25 | 2017-10-18 16:27 |
| Reporter | jacob | Assigned To | henry | ||
| Priority | low | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Platform | GNU/Linux | OS | Ubuntu | OS Version | 14.04 |
| Product Version | dev | ||||
| Fixed in Version | dev | ||||
| Summary | 0002727: mergeMeshes fails when master mesh has zero patches | ||||
| Description | When the "mergeMeshes" utility is applied on empty master mesh (i.e. boundary, points, faces, owner and neighbour contain zero-sized lists) -- which should be an acceptable starting point for merging multiple meshes -- the utility crashes with a segmentation fault. | ||||
| Steps To Reproduce | Create a case with empty mesh (i.e. boundary, points, faces, owner and neighbour contain zero-sized lists) and try to merge any other mesh to this one using "mergeMeshes". | ||||
| Additional Information | The utility wrongly (or unnecessarily) assumes that the master mesh contains at least one patch. A trivial fix is attached to this report. | ||||
| Tags | No tags attached. | ||||
|
|
mergePolyMesh.diff (385 bytes)
--- mergePolyMesh.C 2017-10-18 16:18:01.127756622 +0200
+++ mergePolyMesh-fixed.C 2017-10-16 13:35:07.000000000 +0200
@@ -410,6 +410,7 @@
Info<< "Adding new patches. " << endl;
label endOfLastPatch =
+ patchi == 0 ? 0 :
oldPatches[patchi - 1].start() + oldPatches[patchi - 1].size();
for (; patchi < patchNames_.size(); patchi++)
|
|
|
Thanks for the patch. Resolved by commit 644253773f6edc519859069fe9de2fd93934be79 |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2017-10-18 15:25 | jacob | New Issue | |
| 2017-10-18 15:25 | jacob | File Added: mergePolyMesh.diff | |
| 2017-10-18 16:27 | henry | Assigned To | => henry |
| 2017-10-18 16:27 | henry | Status | new => resolved |
| 2017-10-18 16:27 | henry | Resolution | open => fixed |
| 2017-10-18 16:27 | henry | Fixed in Version | => dev |
| 2017-10-18 16:27 | henry | Note Added: 0008889 |
