Closed Bug 342914 Opened 19 years ago Closed 19 years ago

[FIX]RedirEntry should have a "flags" member instead of two boolean ones

Categories

(Core :: Networking, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: Biesinger, Assigned: bzbarsky)

Details

Attachments

(1 file)

from bug 341313: ------- Bug 341313 Comment #7 From Christian Biesinger (:bi) 2006-06-27 14:49 PDT [reply] ------- (From update of attachment 226425 [details] [diff] [review] [edit]) - { "neterror", "chrome://global/content/netError.xhtml", PR_TRUE } + { "neterror", "chrome://global/content/netError.xhtml", PR_TRUE, PR_TRUE } have you considered making the third field a "flags" field instead, so that the values are more self-documenting and that not all lines have to be changed for a new flag? ------- Bug 341313 Comment #8 From Boris Zbarsky (gone June 26 -- Jul 13) 2006-06-27 15:13 PDT [reply] ------- Hmm... I suppose I could do that, yeah... file a bug and I'll deal in July?
Attached patch PatchSplinter Review
Attachment #227473 - Flags: superreview?(darin)
Attachment #227473 - Flags: review?(cbiesinger)
Attachment #227473 - Flags: superreview?(darin) → superreview+
Summary: RedirEntry should have a "flags" member instead of two boolean ones → [FIX]RedirEntry should have a "flags" member instead of two boolean ones
Attachment #227473 - Flags: review?(cbiesinger) → review+
Status: NEW → RESOLVED
Closed: 19 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: