From 8e49c35c183c0738ddb9b216be3c7725be5d3888 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?B=C3=A5rd=20Skaflestad?= Date: Fri, 29 Mar 2019 14:22:34 +0100 Subject: [PATCH] Create Prop-Func Table: Prune Vacuous Assignment Left over from when the function was extracted from the saturation function code. While here, fix an incorrect comment that mistakenly implied that the primary key of PVTO is pressure rather than the dissolved gas/oil ratio (Rs), and correct a type in 'composition'. --- src/opm/output/eclipse/Tables.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/opm/output/eclipse/Tables.cpp b/src/opm/output/eclipse/Tables.cpp index 216da0cdb..550fd46ee 100644 --- a/src/opm/output/eclipse/Tables.cpp +++ b/src/opm/output/eclipse/Tables.cpp @@ -112,7 +112,6 @@ namespace { const auto numCols = 1 + 2*numDep; auto descr = ::Opm::DifferentiateOutputTable::Descriptor{}; - descr.primID = 0 * numPrim; auto linTable = ::Opm::LinearisedOutputTable { numTab, numPrim, numRows, numCols, fillVal @@ -1679,8 +1678,8 @@ namespace { namespace PVTFunc { auto numActRows = std::size_t{0}; if (primID >= pvto[tableID].size()) { - // Pressure node outside current table's active set. No - // active rows in this subtable. + // Composition node outside current table's active set. + // No active rows in this sub-table. return numActRows; } @@ -1748,7 +1747,7 @@ namespace { namespace PVTFunc { /// composition nodes for all PVT function regions from PVTO (live /// oil with dissolved gas) keyword data. /// - /// \param[in] numCompNodes Number of compositoin nodes to allocate + /// \param[in] numCompNodes Number of composition nodes to allocate /// in the output vector for each oil PVT table. Expected to be /// equal to the number of declared composition nodes in the /// simulation run's TABDIMS keyword (NRPVT, Item 6).