30589 - White space affects <mfenced>

Status: RESOLVED FIXED

(Core :: MathML, defect, P3)

Description

[Paul Gartside]

mfenced will not work without additional space:



Works <mi>n</mi><mi>w</mi> <mfenced open="(" close=")"><mi>Y</mi> </mfenced>

(giving latex equivalent $nw(Y)$)



Doesnt work <mi>w</mi> <mfenced open="(" close=")"><mi>X</mi></mfenced>

(latex equivalent $w(X)$)



The only difference is the extra white space between </mi> and </mfenced>

Comment 1

[rbs]

That's a curious one...

Comment 2

[Paul Gartside]

...and also. Adding the extra space results in spurious commas `,' .

Comment 3

[rbs]

Fixed in my tree. The problem came from the fact that an unrelated temporary
return-value was getting out of nsMathMLContainerFrame::SetInitialChildList():

   nsIFrame* next = mFrames.FirstChild();
   while (next) {
     nsIFrame* child = next;
-    rv = next->GetNextSibling(&next);
+    next->GetNextSibling(&next);
     if (!IsOnlyWhitespace(child)) {
       nsInlineFrame* inlineFrame = nsnull;
-      rv = child->QueryInterface(nsInlineFrame::kInlineFrameCID, 
(void**)&inlineFrame);  
-      if (NS_SUCCEEDED(rv) && inlineFrame) {
+      nsresult res = child->QueryInterface(nsInlineFrame::kInlineFrameCID, 
(void**)&inlineFrame);  
+      if (NS_SUCCEEDED(res) && inlineFrame) {

Comment 4

[rbs]

Code checked in. Marking bug as fixed.

Comment 5

[Hixie (not reading bugmail)]

Testcase is now 404. :-(
Paul: Do you still have a copy of your testcase? If you do, could you verify
the fix? Thanks...