----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37702/#review96120 -----------------------------------------------------------
You can leave off the vararg argument completely as well. The null will be implied. ```java method.invoke(null); ``` is the same as ```java method.invoke(null, (Object[]) null); ``` but in my opinion a little more readable. - Jacob Barrett On Aug. 22, 2015, 9:03 a.m., Anthony Baker wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37702/ > ----------------------------------------------------------- > > (Updated Aug. 22, 2015, 9:03 a.m.) > > > Review request for geode and Kirk Lund. > > > Repository: geode > > > Description > ------- > > Add casts where needed so that invoke(null, null) is now > invoke(null, (Ojbect[]) null). > > > Diffs > ----- > > > gemfire-core/src/test/java/com/gemstone/gemfire/internal/compression/SnappyCompressorJUnitTest.java > 56d647af99178bdb71f6969277cfe3c3f6769ea9 > gemfire-core/src/test/java/dunit/standalone/DUnitLauncher.java > 0f2eebcfc4e17501038f4e5300ea653ebc904b4d > > Diff: https://reviews.apache.org/r/37702/diff/ > > > Testing > ------- > > Ran SnappyCompressorJUnitTest and a DUnit test. > > > Thanks, > > Anthony Baker > >