Closed Bug 331897 Opened 19 years ago Closed 18 years ago

support for validating xsd:double schema type


(Core Graveyard :: XForms, defect)

Not set


(Not tracked)



(Reporter: aaronr, Assigned: sspeiche)



(Keywords: fixed1.8.0.5, fixed1.8.1)


(2 files, 3 obsolete files)

Be nice if we could validate xsd:double schema type.

This actually fails testcase 8.1.7.f in the testsuite.
Attached file testcase
Blocks: 322255
Attached patch patch (obsolete) — Splinter Review
patch + updated test cases.  Verified against attached test case and W3C test suite.

JST review complains about a few things that I think it's just confused about the & operand and String arg types.
Attachment #221706 - Flags: review?(doronr)
Comment on attachment 221706 [details] [diff] [review]

Patch looks good, but the spec says for the lexical space:

"The special values positive and negative infinity and not-a-number have lexical representations INF, -INF and NaN, respectively. Lexical representations for zero may take a positive or negative sign."

Do we need to worry about those?
Attachment #221706 - Flags: review?(doronr) → review+
I hesitate to say this is good enough since the one test case fails.  Though it is unclear if -INF < INF, I would think so.

Let me know if there might be a better way to handle this.
Attachment #221706 - Attachment is obsolete: true
Attachment #221739 - Flags: review?(doronr)
Why is it failing?  I am not sure we need to even consider the 3 text values, do other processors support them?
(In reply to comment #5)
> Why is it failing?  I am not sure we need to even consider the 3 text values,
> do other processors support them?

It is failing since strtod doesn't return appropriate values for INF and -INF (actually tested float and ToFloat() has the same problems).  strtod usually returns 0.0 for both INF and -INF. ToFloat() usually returns garbage for INF and -INF.

X-Smiles uses the Xerces validator, which handles these properly ie -INF < INF, -INF < everything, everything < INF, ... what you'd expect.

So I recommend considering support for double done and followup with with a separate INF/-INF bug for str functions (or should we handle it indirectly in our processing of infinity).  Note, after saying all of this...I can't possibly think that anyone is using infinity but you never know.
Don't know if you want these committed but thought it might be helpful.  Updated to test with INF/-INF
Comment on attachment 221739 [details] [diff] [review]
patch2 - with support for INF/-INF/NaN

Yeah, lets go with this and file a bug about INF/-INF.
Attachment #221739 - Flags: review?(doronr) → review+
Attachment #221739 - Flags: review?(allan)
Comment on attachment 221811 [details] [diff] [review]
patch for schema test suite, shows INF edge cases

obsoleting, will move to bug regarding INF/-INF
Attachment #221811 - Attachment is obsolete: true
Comment on attachment 221739 [details] [diff] [review]
patch2 - with support for INF/-INF/NaN

Isn't this Doron-land? That said, the code looks sane, except for:

>-  return (pEnd != aString);
>+  // If end pointer hasn't moved, then there was an error

Either the code or the comment is wrong then.

>+  if (pEnd == aString) {
>+    NS_NAMED_LITERAL_CSTRING(temp, aString);
>+    // doubles may be INF, -INF or NaN
>+    if (!temp.Equals(NS_LITERAL_CSTRING("INF")) &&
>+        !temp.Equals(NS_LITERAL_CSTRING("-INF")) &&
>+        !temp.Equals(NS_LITERAL_CSTRING("NaN"))) {
>+      isValid = PR_FALSE;
>+    }
>+  }
>+  return isValid;

With that fixed, r=me
Attachment #221739 - Flags: review?(allan) → review+
Doron allowed me to enter Doron-land.

Fixed comment, ready for checkin.
Attachment #221739 - Attachment is obsolete: true
(In reply to comment #11)
> Created an attachment (id=222030) [edit]
> patch 3, with comment fixed
> Doron allowed me to enter Doron-land.

:) I meant: r=doronr, and be done with it?
Assignee: doronr → sspeiche
checked into trunk for sspeiche
Closed: 18 years ago
Resolution: --- → FIXED
Whiteboard: xf-to-branch
Doron-land is a happy place with clowns and free baloons.
Blocks: 331209
Keywords: fixed1.8.1
Keywords: fixed1.8.0.5
Whiteboard: xf-to-branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.