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., Antho

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

2015-08-24 Thread Jacob Barrett
> On Aug. 22, 2015, 9:39 a.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 li

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 li

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 w

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 Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37702/ --- Review request for geode and Kirk Lund. Repository: geode Description ---