I've seen such meaningless code in many other places and turned them
into asserts. Please do so instead of removing them.
On 11/15/2012 11:13 AM, Dan Berindei wrote:
I'm guessing the author's intention was to make them both asserts.
În data de 15.11.2012 07:35, "Navin Surtani" <[email protected]
<mailto:[email protected]>> a scris:
I noticed a couple of unused returns in the test class mentioned
in the subject. The specific code lines can be found here [1].
If there aren't any objections I'm going to remove these calls as
I can't see why that code is there. The returned boolean type
isn't being used for any checks. Well, it's not being used at all. :-)
Thanks!
------------------------
Navin Surtani
Software Engineer
JBoss SET
JBoss EAP
Twitter: @navssurtani
[1] -
https://github.com/infinispan/infinispan/blob/master/core/src/test/java/org/infinispan/tx/recovery/RecoveryWithDefaultCacheDistTest.java#L61
_______________________________________________
infinispan-dev mailing list
[email protected] <mailto:[email protected]>
https://lists.jboss.org/mailman/listinfo/infinispan-dev
_______________________________________________
infinispan-dev mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/infinispan-dev
_______________________________________________
infinispan-dev mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/infinispan-dev
