Closed Bug 1212462 Opened 4 years ago Closed 4 years ago

Rename nsRefPtr<T> to RefPtr<T> due to mozilla-central Bug 1207245 (get rid of nsRefPtr / RefPtr split)

Categories

(MailNews Core :: Backend, defect)

defect
Not set

Tracking

(firefox44 unaffected, seamonkey2.41 fixed)

RESOLVED FIXED
Thunderbird 44.0
Tracking Status
firefox44 --- unaffected
seamonkey2.41 --- fixed

People

(Reporter: philip.chee, Assigned: frg)

References

Details

Attachments

(1 file)

No description provided.
Just used the script from Bug 1207245. is it ok to include the other directories here too?

>> find . -name '*.cpp' -o -name '*.cc' -o -name '*.h' -o -name '*.mm' -o -name >> '*.idl'| \
>>    xargs perl -p -i -e '
>> s/nsRefPtr\.h/RefPtr\.h/g; # handle includes
>>  s/nsRefPtr ?</RefPtr</g;   # handle declarations and variables
>> '
Comment on attachment 8675424 [details] [diff] [review]
1212462-nsrefptr.patch

(In reply to Frank-Rainer Grahl from comment #1)
> Just used the script from Bug 1207245. is it ok to include the other
> directories here too?

Yes. That's fine. With your patch SeaMonkey compiles. I'm going to push this since it fixes the current bustage on comm-central
rs=Ratty a=Ratty bustage fix
Attachment #8675424 - Flags: review+
> remote: *************************** ERROR ***************************
> remote: Push rejected because the following IDL interfaces were
> remote: modified without changing the UUID:
> remote:   - nsIMsgSend in changeset 7e5b1ec4f9f2
> remote: 
> remote: To update the UUID for all of the above interfaces and their
> remote: descendants, run:
> remote:   ./mach update-uuids nsIMsgSend
> remote: 
> remote: If you intentionally want to keep the current UUID, include
> remote: 'IGNORE IDL' in the commit message.
> remote: *************************************************************
Oops. Forgot to change the UUID. I've done that for you.

Pushed to comm-central http://hg.mozilla.org/comm-central/rev/9579c61e62f5
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 44.0
Assignee: nobody → frgrahl
You need to log in before you can comment on or make changes to this bug.