tuxji commented on a change in pull request #561:
URL: https://github.com/apache/daffodil/pull/561#discussion_r633662278
##########
File path:
daffodil-runtime1/src/main/scala/org/apache/daffodil/processors/dfa/Rules.scala
##########
@@ -104,7 +106,9 @@ abstract class State(states: => ArrayBuffer[State]) extends
Serializable {
val res =
if (pTerm0.isInstanceOf[WSPStarState]) {
val wspStar = pTerm0.asInstanceOf[WSPStarState]
- if (wspStar.checkMatch(charIn)) { true } // Was a space
+ if (wspStar.checkMatch(charIn)) {
+ true
+ } // Was a space
Review comment:
Comment should be moved to previous line and push previous line below
it, e.g.,
```scala
// Was a space.
//
true
```
This makes this branch look similar to the following branches and therefore
makes the entire expression easier to read.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
