Github user mbeckerle commented on a diff in the pull request:

    https://github.com/apache/incubator-daffodil/pull/5#discussion_r149844708
  
    --- Diff: 
daffodil-core/src/main/scala/edu/illinois/ncsa/daffodil/dsom/TermEncodingMixin.scala
 ---
    @@ -114,9 +114,6 @@ trait TermEncodingMixin extends KnownEncodingMixin { 
self: Term =>
             mg.hasNoSkipRegions &&
               hasTextAlignment
           }
    -      case gr: GroupRef => {
    --- End diff --
    
    Add scaladoc to this trait to explain what its role is. Maybe rename to 
something more specific. 


---

Reply via email to