mbeckerle commented on a change in pull request #150: Removed use of
ThreadLocal for DFA Register Pool.
URL: https://github.com/apache/incubator-daffodil/pull/150#discussion_r240830886
##########
File path:
daffodil-runtime1/src/main/scala/org/apache/daffodil/processors/dfa/TextParser.scala
##########
@@ -23,30 +23,30 @@ import org.apache.daffodil.processors.TermRuntimeData
import org.apache.daffodil.processors.DelimiterIterator
import org.apache.daffodil.io.DataInputStream
import org.apache.daffodil.io.FormatInfo
+import org.apache.daffodil.processors.parsers.PState
-class TextParser(
- override val context: TermRuntimeData)
+class TextParser(override val context: TermRuntimeData)
extends DFAParser {
override lazy val name: String = "TextParser"
override lazy val info: String = "" // Nothing additional to add here
- def parse(finfo: FormatInfo, input: DataInputStream, delimIter:
DelimiterIterator, isDelimRequired: Boolean): Maybe[ParseResult] = {
+ def parse(finfo: PState, input: DataInputStream, delimIter:
DelimiterIterator, isDelimRequired: Boolean): Maybe[ParseResult] = {
Review comment:
change name here too
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
