Re: Review Request 37702: GEODE-264: Fix varargs compiler warnings

2015-09-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37702/#review97333 --- Ship it! Ship It! - Kirk Lund On Aug. 22, 2015, 4:03 p.m.,

Re: Review Request 37702: GEODE-264: Fix varargs compiler warnings

2015-08-23 Thread Anthony Baker
On Aug. 22, 2015, 4:39 p.m., Jacob Barrett wrote: 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.

Re: Review Request 37702: GEODE-264: Fix varargs compiler warnings

2015-08-22 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37702/ --- (Updated Aug. 22, 2015, 4:03 p.m.) Review request for geode and Kirk Lund.

Re: Review Request 37702: GEODE-264: Fix varargs compiler warnings

2015-08-22 Thread Jacob Barrett
--- 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