Closed
Bug 690700
Opened 13 years ago
Closed 13 years ago
Remove PR_TRUE/PR_FALSE from widget/src/gtk2
Categories
(Core :: Widget: Gtk, defect)
Tracking
()
RESOLVED
FIXED
mozilla10
People
(Reporter: masayuki, Assigned: masayuki)
References
Details
(Whiteboard: [inbound])
Attachments
(2 files)
177.73 KB,
patch
|
karlt
:
review+
|
Details | Diff | Splinter Review |
1.14 KB,
patch
|
karlt
:
review+
|
Details | Diff | Splinter Review |
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=505027082410 ([ ,({])PR_TRUE([ ,);}\n]) and ([ ,({])PR_FALSE([ ,);}\n]) are replaced with true and false. And I fixed two lines which is in string for log (nsDragService::GetData()) manually.
Attachment #563698 -
Flags: review?(karlt)
Comment 1•13 years ago
|
||
Instead of updating the MAKE_PR_BOOL macro, it looks like you can just remove it entirely. Not only is it generally obsolete but it appears to only be defined and not used. http://mxr.mozilla.org/mozilla-central/search?string=MAKE_PR_BOOL
Assignee | ||
Comment 3•13 years ago
|
||
NOTE: both MAKE_PR_BOOL have no user.
Updated•13 years ago
|
Attachment #564014 -
Flags: review?(karlt) → review+
Comment 4•13 years ago
|
||
Comment on attachment 563698 [details] [diff] [review] Patch >- gtk_file_chooser_set_do_overwrite_confirmation(GTK_FILE_CHOOSER(file_chooser), PR_TRUE); >+ gtk_file_chooser_set_do_overwrite_confirmation(GTK_FILE_CHOOSER(file_chooser), true); >- GdkPixbuf* pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB, PR_TRUE, 8, >+ GdkPixbuf* pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB, true, 8, These function expect gboolean, so please use "TRUE" here. >- gboolean value = PR_FALSE; >+ gboolean value = false; Similarly, "FALSE" here.
Attachment #563698 -
Flags: review?(karlt) → review+
Assignee | ||
Comment 5•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/696394093f34 https://hg.mozilla.org/integration/mozilla-inbound/rev/450f539a3dab
Flags: in-testsuite-
Whiteboard: [inbound]
Comment 6•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/450f539a3dab https://hg.mozilla.org/mozilla-central/rev/696394093f34 Please stop doing PR_TRUE/PR_FALSE convertions till the PRBool regression is figured out (see the topic in mozilla.dev.planning)
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
Comment 7•13 years ago
|
||
Hi, We are looking into why tp5 for Win7 could have become bimodal after Oct. 3rd in bug 697092. After much digging up the changeset 696394093f34 has become our main suspect (we're are narrowing down the variables). Is there any way you think this patch could have caused this? Any help will be very much appreciated as this type of distributions prevents anyone to determine when new performance regressions happen. Thanks a lot! http://graphs-new.mozilla.org/graph.html#tests=[[89,63,12],[115,1,12],[115,63,12]]&sel=1314736531811,1322512531811&displayrange=90&datatype=running
Assignee | ||
Comment 8•13 years ago
|
||
Armen: This bug just replaced PR_TRUE/PR_FALSE with true/false (or TRUE/FALSE for gboolean).
Comment 9•13 years ago
|
||
Answered in bug 697092 comment 13.
You need to log in
before you can comment on or make changes to this bug.
Description
•