Add Ice 3.5.1 jars and ivy dependencies
lib: Add generated omero-shares-3.5.1.jar
lib: Add omero-icemock-3.5.1.jar
build: release-javadoc: Exclude omero client jars

These jars break with javadoc from java 1.7 due to containing
source code as well as classes.  The reasons why this occurs
are not clear, but excluding them works around the immediate
breaking of the build.  The breakage occurs with the Ice 3.5.1
jars, but given that the 3.5.0 (and other) jars also contain
essentially similar content, it's not clear what the trigger
for the breakage is.  It could be a javadoc bug.
flake8 cleanup of some test files
Minor typo fixes
testGetTile() checks against getPlane(). See #11315
Update test for py.test
Add swapIfRequired to getTileDirect (Fix #10581)

The other `get*Direct` methods all contain a call to
`reader.swapIfRequired()`. The same logic in the non-
direct methods is handled by a call to `setOrder` on
the newly created `PixelData` object.
Prevent double swapping of tile bytes

Each of the `get*Direct` methods now call
swapIfRequired. Non-direct methods should
not call them since double swapping could
occur.
Modify testGetTile to use gatewaywrapper from fixture.
Minor refactoring of BFPPB for testing

In order to be able to intercept (and block) certain
methods of BFPPB for concurrency testing methods have
been changed to `protected` and an `init()` method has
been introduced to allow a ctor with less logic.
Passing test with intermediate fix

The change (basically to not lock on read) makes
this test pass where before it was failing (when
`Thread.sleep()` was artificially introduced for
calls to releaseLock.

It's unknown if this will be sufficiently safe
going forward.
Initial error handling of tile image loading in PanoJS See #11709
Restrict event logs on chgrp
Prevent array copy of IDs
Add hot-bioformats command
Move CHGRP event log to ChgrpI class
Explain non-ID copying
Clean up code.