Note: There are a few cases of duplicates in user autocompletion which are being worked on.

nsRange::GetNodeLength declares len as unsigned and uses it as signed

RESOLVED FIXED

Status

()

Core
DOM: Core & HTML
RESOLVED FIXED
15 years ago
4 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

15 years ago
653 PRInt32 nsRange::GetNodeLength(nsIDOMNode *aNode)
 654 {
 659   PRUint32 len = -1;
 677   return len;
 678 }

This triggers warnings and is silly.
timeless:  fixed by jst?
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&root=/cvsroot&subdir=mozilla/content/base/src&command=DIFF_FRAMESET&file=nsRange.cpp&rev2=1.170&rev1=1.169
QA Contact: desale → traversal-range
This was evidently fixed long ago.  Today's nsRange::GetNodeLength returns a PRUint32.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Component: DOM: Traversal-Range → DOM: Core & HTML
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.