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)
Core
Networking
Tracking
()
RESOLVED
FIXED
People
(Reporter: Biesinger, Assigned: bzbarsky)
Details
Attachments
(1 file)
5.67 KB,
patch
|
Biesinger
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
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?
![]() |
Assignee | |
Comment 1•19 years ago
|
||
Attachment #227473 -
Flags: superreview?(darin)
Attachment #227473 -
Flags: review?(cbiesinger)
Updated•19 years ago
|
Attachment #227473 -
Flags: superreview?(darin) → superreview+
![]() |
Assignee | |
Updated•19 years ago
|
Summary: RedirEntry should have a "flags" member instead of two boolean ones → [FIX]RedirEntry should have a "flags" member instead of two boolean ones
Reporter | ||
Updated•19 years ago
|
Attachment #227473 -
Flags: review?(cbiesinger) → review+
![]() |
Assignee | |
Updated•19 years ago
|
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.
Description
•