Closed Bug 311036 Opened 19 years ago Closed 19 years ago

String offset problem in the XForms XPath engine

Categories

(Core Graveyard :: XForms, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: allan, Assigned: allan)

References

()

Details

(Keywords: fixed1.8.0.2, fixed1.8.1)

Attachments

(2 files)

*sigh* wrong and stupid fix in bug 277929 makes a lot of XPath functions fail.
Attached file Testcase
This testcase uses following-sibling:: and fails
Attached patch PatchSplinter Review
Fixes the offset-error, instead of the original "insert a space before name"
*sigh*. (also removes the unnecessary else statements...)

I also added a warning so it's easier to detect that the scanner returns an
error token.
Attachment #198430 - Flags: review?(smaug)
Attachment #198430 - Flags: review?(smaug)
Attachment #198430 - Flags: review?(doronr)
Attachment #198430 - Flags: review+
Attachment #198430 - Flags: review?(doronr) → review+
This was checked in to trunk quite a while ago
Status: NEW → ASSIGNED
Whiteboard: xf-to-branch
checked into MOZILLA_1_8_BRANCH via bug 323691.  Leaving open for now until it gets into 1.8.0
Whiteboard: xf-to-branch
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Keywords: fixed1.8.0.2
Resolution: --- → FIXED
verified fixed in MOZILLA_1_8_BRANCH
Keywords: fixed1.8.1
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: