Clarify retained network THP limit comments

Author: GitPaean

opm/input/eclipse/Schedule/Action/SimulatorUpdate.hpp

@@ -72,12 +72,12 @@ struct SimulatorUpdate
     /// its internal notion of the connection transmissibility factors.
     std::unordered_set<std::string> welpi_wells{};
 
-    /// Wells whose THP limit and/or VFP table has been (re)specified, by
+    /// Wells whose THP limit and/or VFP table has been (re)specified by
     /// WCONPROD or WCONHIST for the named wells, or by WELTARG with THP or
-    /// VFP control. Unlike affected_wells, the wells are included also when
-    /// the entered values are unchanged, since re-specifying the THP limit
-    /// cancels a limit imposed dynamically by the simulator (e.g. by
-    /// network balancing). See also the WELL_THP_UPDATE schedule event.
+    /// VFP control. Unlike affected_wells, this set also includes wells for
+    /// which the entered values are unchanged, since re-specifying either
+    /// input cancels a THP limit imposed dynamically by the simulator (e.g.
+    /// by network balancing). See also the WELL_THP_UPDATE schedule event.
     std::unordered_set<std::string> thp_respec_wells{};
 
     /// New well connections created as a result of a geomechanical

opm/input/eclipse/Schedule/Events.hpp

@@ -114,11 +114,11 @@ namespace Opm
             /// the recorded schedule state).
             REQUEST_OPEN_COMPLETION = (UINT64_C(1) << 24),
 
-            /// The well THP limit and/or VFP table has been (re)specified,
+            /// The well THP limit and/or VFP table has been (re)specified
             /// by WCONPROD or WCONHIST for the named wells, or by WELTARG
             /// with THP or VFP control.  Unlike PRODUCTION_UPDATE, the event
-            /// is triggered also when the entered values are unchanged,
-            /// since re-specifying the THP limit cancels a limit imposed
+            /// is also triggered when the entered values are unchanged,
+            /// since re-specifying either input cancels a THP limit imposed
             /// dynamically by the simulator (e.g. by network balancing).
             WELL_THP_UPDATE = (UINT64_C(1) << 25),
         };

opm/input/eclipse/Schedule/Well/WellKeywordHandlers.cpp

@@ -188,7 +188,7 @@ void handleWCONHIST(HandlerContext& handlerContext)
                 update_well = true;
             }
 
-            // The VFP table (item 8) is re-specified by every WCONHIST,
+            // The VFP table (item 7) is re-specified by every WCONHIST,
             // whether or not any value changed.
             handlerContext.state().events().addEvent( ScheduleEvents::WELL_THP_UPDATE );
             handlerContext.state().wellgroup_events().addEvent( well2.name(), ScheduleEvents::WELL_THP_UPDATE);