Last Comment Bug 739124 - Remove unused nsCharsetProber::SetOpion and overrides
: Remove unused nsCharsetProber::SetOpion and overrides
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Internationalization (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla14
Assigned To: Ehren Metcalfe
:
: Makoto Kato [:m_kato]
Mentors:
Depends on:
Blocks: deadcode
  Show dependency treegraph
 
Reported: 2012-03-25 21:10 PDT by Ehren Metcalfe
Modified: 2012-08-09 08:21 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (13.58 KB, patch)
2012-03-25 21:11 PDT, Ehren Metcalfe
smontagu: review+
Details | Diff | Splinter Review

Description Ehren Metcalfe 2012-03-25 21:10:11 PDT
The methods have no body but this kills a bunch of vtable entries http://mxr.mozilla.org/mozilla-central/search?string=SetOpion
Comment 1 Ehren Metcalfe 2012-03-25 21:11:49 PDT
Created attachment 609201 [details] [diff] [review]
patch
Comment 2 Simon Montagu :smontagu 2012-03-26 08:57:10 PDT
Comment on attachment 609201 [details] [diff] [review]
patch

Review of attachment 609201 [details] [diff] [review]:
-----------------------------------------------------------------

What an Opion is I'll never know ;-)
Comment 4 Ed Morley [:emorley] 2012-03-27 05:27:19 PDT
https://hg.mozilla.org/mozilla-central/rev/ec5ee2b448c3

Note You need to log in before you can comment on or make changes to this bug.