- About Scala
- Documentation
- Code Examples
- Software
- Scala Developers
[scala-bts] #2124: Xml RuleTransformer incorrectly transforms simple XML fragment
Sun, 2009-07-05, 11:47
-------------------------+--------------------------------------------------
Reporter: BenHutchison | Owner: scala-xml_team
Type: defect | Status: new
Priority: normal | Component: XML support
Keywords: |
-------------------------+--------------------------------------------------
Expected:
Actual:
Test Code:
import scala.xml._
import scala.xml.transform._
object TransformBug {
val sampleXml =
def main(args: scala.Array[String]) {
println(new RuleTransformer(new RewriteRule {
override def transform(n: Node): NodeSeq = {
val result = n match {
case {_*} =>
case n => n
}
// println ("Rewriting '" +n+ "' to: '" + result+ "'")
result
}
}).transform(sampleXml))
}
}
Sun, 2009-07-05, 16:57
#2
Re: [scala-bts] #2124: Xml RuleTransformer incorrectly transform
-----------------------------+----------------------------------------------
Reporter: BenHutchison | Owner: scala-xml_team
Type: defect | Status: closed
Priority: normal | Component: XML support
Version: Unreleased-2.8.x | Resolution: fixed
Keywords: |
-----------------------------+----------------------------------------------
Changes (by extempore):
* cc: paulp@… (added)
* status: new => closed
* version: => Unreleased-2.8.x
* resolution: => fixed
Comment:
In another fine indictment of index-based iteration...
{{{
/** Returns a new node buffer with the first pos elements
* from ns.
...
var j = 0; while (j < pos-1) {
}}}
Fixed in r18210.
-------------------------+--------------------------------------------------
Reporter: BenHutchison | Owner: scala-xml_team
Type: defect | Status: new
Priority: normal | Component: XML support
Keywords: |
-------------------------+--------------------------------------------------
Comment(by BenHutchison):
A debugging session suggests that the problem happens on line 95 of
BasicTranformer:
nb ++= (n2:Iterable[xml.Node])
'nb' includes the element beforehand, but loses it after.