Don't use unnecessary PR* typedefs

RESOLVED FIXED in mozilla17

Status

()

RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

Trunk
mozilla17
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(7 attachments, 1 obsolete attachment)

(Assignee)

Description

7 years ago
There is very little point in using, say |typedef size_t PRSize;| when we can use size_t directly.
(Assignee)

Comment 1

7 years ago
Created attachment 648992 [details] [diff] [review]
Part a: PRPtrdiff
Attachment #648992 - Flags: review?(benjamin)
(Assignee)

Comment 2

7 years ago
Created attachment 648993 [details] [diff] [review]
Part b: PRIntn
Attachment #648993 - Flags: review?(benjamin)
(Assignee)

Comment 3

7 years ago
Created attachment 648994 [details] [diff] [review]
Part c: PRUintn
Attachment #648994 - Flags: review?(benjamin)
(Assignee)

Comment 4

7 years ago
Created attachment 648995 [details] [diff] [review]
Part d: PRFloat64
Attachment #648995 - Flags: review?(benjamin)
(Assignee)

Comment 5

7 years ago
Created attachment 648996 [details] [diff] [review]
Part e: PRSize
Attachment #648996 - Flags: review?(benjamin)
(Assignee)

Comment 6

7 years ago
Created attachment 648997 [details] [diff] [review]
Part f: PRBool / PRPackedBool

Turns out those keep sneaking in.
Attachment #648997 - Flags: review?(benjamin)
(Assignee)

Comment 7

7 years ago
Created attachment 648998 [details]
Sample script

Parts b-e were generated with a shell script stolen from the prbool bug like this one.

Updated

7 years ago
Attachment #648992 - Flags: review?(benjamin) → review+

Updated

7 years ago
Attachment #648993 - Flags: review?(benjamin) → review+

Comment 8

7 years ago
Comment on attachment 648994 [details] [diff] [review]
Part c: PRUintn

I don't think we should do this for the cases which are using PR_NewThreadPrivateIndex, which still uses PRUintn*.
Attachment #648994 - Flags: review?(benjamin) → review-

Updated

7 years ago
Attachment #648995 - Flags: review?(benjamin) → review+

Updated

7 years ago
Attachment #648996 - Flags: review?(benjamin) → review+

Updated

7 years ago
Attachment #648997 - Flags: review?(benjamin) → review+
(Assignee)

Comment 9

7 years ago
Created attachment 650211 [details] [diff] [review]
Part c: PRUintn (v2)

I think I caught them all.
Attachment #648994 - Attachment is obsolete: true
Attachment #650211 - Flags: review?(benjamin)
Comment on attachment 650211 [details] [diff] [review]
Part c: PRUintn (v2)

I understand this to have been subsumed by bug 579517, which I hope to get to shortly.
Attachment #650211 - Flags: review?(benjamin)
(Assignee)

Comment 16

7 years ago
The parts we're going to fix in this bug are fixed, then.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.