rzwitserloot/lombok — pull request #49 by Maaartinus (2014-06-01 19:33:17+00:00)
Title: Fixed version of 842a475: Remove a wrong test for copyAnnotations return...

Concerning 842a475: You've misunderstood me. My point was that copyAnnotations *itself* can return null and then you'll get an NPE in testing for zero length. But you were right with reverting it, as I made a mistake: copyAnnotations could return *both* null and a zero-length array! So a null-safe emptiness test would be needed. That's plain wrong.

So I sort of un-reverted it and made `copyAnnotations` never return a zero-length array, so we don't need the test.

IMHO every method should do exactly what's needed and Eclipse being insane should not make us insane.