One more thing I forgot to mention: Fowler suggests:

Don't inline if subclasses override the method; they cannot override a method that isn't there (Fowler p. 118)

In other words, if the method has been factored out to provide an easy override hook, leave it alone. This can present challenges if you have a large code base but I usually find that:

find . -exec grep that_method {} \; -print

does the trick nicely for finding such things

perl -e 'split//,q{john hurl, pest caretaker}and(map{print @_[$_]}(joi
+n(q{},map{sprintf(qq{%010u},$_)}(2**2*307*4993,5*101*641*5261,7*59*79
+*36997,13*17*71*45131,3**2*67*89*167*181))=~/\d{2}/g));'
[download]