Made better message for USBR_UNNECESSARY_STORE_BEFORE_RETURN

Gave an example
Various cleanups

Added Overrides, made a final else clause for RegisterUtils and used
some Types's signatures directly instead of always invoking
getSignature.
Fixed bad spacing in </br>
Merge branch 'forpullReq' into outgoingChanges
merged in upstream/master
Merge branch 'forpullReq' into outgoingChanges
An embarrassing amount of </br>

But, apparently the FindBugs html engine makes them look fine, thus I
didn't notice.
Added web javadocs for BCEL

The sources fetched by yank are no good.
Some development steps
Merge remote-tracking branch 'upstream/master' into outgoingChanges_docs_for_devs

Conflicts:
	src/com/mebigfatguy/fbcontrib/collect/MethodInfo.java
	src/com/mebigfatguy/fbcontrib/debug/OCSDebugger.java
Finished first round of dev

I still need to link to some dev resources.
More verbose OCP description
Merge remote-tracking branch 'upstream/master' into outgoingChanges

Conflicts:
	src/com/mebigfatguy/fbcontrib/collect/MethodInfo.java
	src/com/mebigfatguy/fbcontrib/debug/OCSDebugger.java
Merge errata

Also cleaned up the OCP message
Added links for external guides
Floating download
Merge branch 'outgoingChanges' into outgoingChanges_docs_for_devs
Final touches

Cleaned up downloads and clarified where you should build it.
Updated default encoding message

This reflects StandardCharsets in 1.7
Replicated test case for OCP false negative

See kjlubick/fb-contrib/issues/20 for more details.
Some investigation shows that simply printing the stack trace in the catch
clause will not invoke the false negative