Closed Bug 581648 Opened 14 years ago Closed 14 years ago

Useless warning in GetReversedHostName

Categories

(Toolkit :: Places, defect)

Product:
Toolkit
Component:
Places
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla2.0b3

People

(Reporter: mak, Assigned: mak)

Details

Attachments

(1 file, 1 obsolete file)

patch v1.1
734 bytes, patch
sdwilsh
: review+
sdwilsh
: approval2.0+
Details | Diff | Splinter Review 
Not all URIs have a valid host.
Attached patch patch v1.0 (obsolete) — — Splinter Review 
This probably does not need approval to land, pretty trivial. if you ant to land for me I'll appreciate it. thx.
Attachment #460013 - Flags: review?(sdwilsh)
Attached patch patch v1.1 — — Splinter Review 
ehr sorry, better forwarding error :)
Attachment #460013 - Attachment is obsolete: true
Attachment #460014 - Flags: review?(sdwilsh)
Attachment #460013 - Flags: review?(sdwilsh)
Comment on attachment 460014 [details] [diff] [review]
patch v1.1

r=sdwilsh
Attachment #460014 - Flags: review?(sdwilsh)
Attachment #460014 - Flags: review+
Attachment #460014 - Flags: approval2.0+
http://hg.mozilla.org/mozilla-central/rev/af9b33fcb752
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: