Shanedell commented on code in PR #875:
URL: https://github.com/apache/daffodil/pull/875#discussion_r1025553710
##########
daffodil-runtime1/src/main/scala/org/apache/daffodil/processors/DataProcessor.scala:
##########
@@ -128,24 +120,20 @@ object DataProcessor {
* back-end runtime.
*/
class DataProcessor private (
- //
- // Once the deprecated API goes away, these var can become val.
- //
- var ssrd: SchemaSetRuntimeData,
- var tunables: DaffodilTunables, // Compiler-set tunables
- var variableMap: VariableMap,
+ val ssrd: SchemaSetRuntimeData,
+ val tunables: DaffodilTunables, // Compiler-set tunables
+ val variableMap: VariableMap,
//
// The below do not need to be transient
// because this object itself isn't serialized. A SerializableDataProcessor
is.
// The values these will have (since this is a base class) are the correct
default values that we want
// back when the object is re-initialized.
//
- protected var areDebugging : Boolean,
- protected var optDebugger : Option[Debugger],
+ protected val areDebugging : Boolean,
+ protected val optDebugger : Option[Debugger],
var validationMode: ValidationMode.Type,
private var externalVars: Queue[Binding])
Review Comment:
@stevedlawrence That look good? So since I have two approvals, I should wait
till the CI passed, then squash my commits, push up the squashed commit and
then merge once CI passes?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
