Closed Bug 441339 Opened 16 years ago Closed 16 years ago

Simplify number-optional-number parsing

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: longsonr, Assigned: longsonr)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
      No description provided.
Attachment #326334 - Flags: superreview?(roc)
Attachment #326334 - Flags: review?(roc)
Nice!

-    ReportAttributeParseFailure(GetOwnerDoc(), aAttribute, aValue);

We're losing these error reports. We really need to keep them.
I don't think we are. The caller reports the errors using its existing mechanism. Not quite enough context to include it but it goes like this...

    if (foundMatch) {
      if (NS_FAILED(rv)) {
        ReportAttributeParseFailure(GetOwnerDoc(), aAttribute, aValue);
        return PR_FALSE;
      }
      aResult.SetTo(aValue);
      return PR_TRUE;
    }
Comment on attachment 326334 [details] [diff] [review]
patch

ok!

Are there any tests that cover this?
Attachment #326334 - Flags: superreview?(roc)
Attachment #326334 - Flags: superreview+
Attachment #326334 - Flags: review?(roc)
Attachment #326334 - Flags: review+
We have partial coverage in that some of the reftests you wrote have number-optional-number types. I'll expand the SVG type mochitest when I do my next type conversion to do expanded number-optional-number testing.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: