From 1d517e72ce3c7b06ea5083f9925e2e6d25abb664 Mon Sep 17 00:00:00 2001 From: mattijs Date: Thu, 9 Jan 2014 12:14:57 +0000 Subject: [PATCH 01/13] BUG: dimensionSet: avoid symbolic output in binary since ^ not parsed correctly --- src/OpenFOAM/dimensionSet/dimensionSetIO.C | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/OpenFOAM/dimensionSet/dimensionSetIO.C b/src/OpenFOAM/dimensionSet/dimensionSetIO.C index e3cbb64fd5..b7a2cbc5ae 100644 --- a/src/OpenFOAM/dimensionSet/dimensionSetIO.C +++ b/src/OpenFOAM/dimensionSet/dimensionSetIO.C @@ -2,7 +2,7 @@ ========= | \\ / F ield | OpenFOAM: The Open Source CFD Toolbox \\ / O peration | - \\ / A nd | Copyright (C) 2011-2013 OpenFOAM Foundation + \\ / A nd | Copyright (C) 2011-2014 OpenFOAM Foundation \\/ M anipulation | ------------------------------------------------------------------------------- License @@ -652,7 +652,7 @@ Foam::Ostream& Foam::dimensionSet::write os << token::BEGIN_SQR; - if (writeUnits.valid()) + if (writeUnits.valid() && os.format() == IOstream::ASCII) { scalarField exponents(dimensionSet::nDimensions); for (int d=0; d Date: Thu, 9 Jan 2014 12:56:10 +0000 Subject: [PATCH 02/13] ENH: solidBodyMotionFvMesh - Added check to ensure cellZone is set --- .../solidBodyMotionFvMesh.C | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/src/dynamicFvMesh/solidBodyMotionFvMesh/solidBodyMotionFvMesh.C b/src/dynamicFvMesh/solidBodyMotionFvMesh/solidBodyMotionFvMesh.C index 5bb0617b06..44a1a2c4f2 100644 --- a/src/dynamicFvMesh/solidBodyMotionFvMesh/solidBodyMotionFvMesh.C +++ b/src/dynamicFvMesh/solidBodyMotionFvMesh/solidBodyMotionFvMesh.C @@ -2,7 +2,7 @@ ========= | \\ / F ield | OpenFOAM: The Open Source CFD Toolbox \\ / O peration | - \\ / A nd | Copyright (C) 2011-2012 OpenFOAM Foundation + \\ / A nd | Copyright (C) 2011-2014 OpenFOAM Foundation \\/ M anipulation | ------------------------------------------------------------------------------- License @@ -94,10 +94,23 @@ Foam::solidBodyMotionFvMesh::solidBodyMotionFvMesh(const IOobject& io) if (cellZoneName != "none") { - zoneID_ = cellZones().findZoneID(cellZoneName); Info<< "Applying solid body motion to cellZone " << cellZoneName << endl; + zoneID_ = cellZones().findZoneID(cellZoneName); + + if (zoneID_ == -1) + { + FatalErrorIn + ( + "solidBodyMotionFvMesh::solidBodyMotionFvMesh(const IOobject&)" + ) + << "Unable to find cellZone " << cellZoneName + << ". Valid celLZones are:" + << cellZones().names() + << exit(FatalError); + } + const cellZone& cz = cellZones()[zoneID_]; From a1e9e5187014bb61842eebfbac79d57e0f7f8376 Mon Sep 17 00:00:00 2001 From: andy Date: Thu, 9 Jan 2014 12:57:06 +0000 Subject: [PATCH 03/13] BUG: LocalInteraction - do not reset list to zero --- .../PatchInteractionModel/LocalInteraction/LocalInteraction.C | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/lagrangian/intermediate/submodels/Kinematic/PatchInteractionModel/LocalInteraction/LocalInteraction.C b/src/lagrangian/intermediate/submodels/Kinematic/PatchInteractionModel/LocalInteraction/LocalInteraction.C index 47e73f1818..77a6debde7 100644 --- a/src/lagrangian/intermediate/submodels/Kinematic/PatchInteractionModel/LocalInteraction/LocalInteraction.C +++ b/src/lagrangian/intermediate/submodels/Kinematic/PatchInteractionModel/LocalInteraction/LocalInteraction.C @@ -2,7 +2,7 @@ ========= | \\ / F ield | OpenFOAM: The Open Source CFD Toolbox \\ / O peration | - \\ / A nd | Copyright (C) 2011-2013 OpenFOAM Foundation + \\ / A nd | Copyright (C) 2011-2014 OpenFOAM Foundation \\/ M anipulation | ------------------------------------------------------------------------------- License @@ -298,7 +298,7 @@ void Foam::LocalInteraction::info(Ostream& os) this->getModelProperty("massStick", mps0); // accumulate current data - labelList npe(nEscape_, 0); + labelList npe(nEscape_); Pstream::listCombineGather(npe, plusEqOp