Code bugs.

[RusTit]

At first i thought that something is broken, but after some tests I realized that everything work (flower image sizes). So calculation can be removed.
Fuchsia has one section.

Binnie/botany/src/main/java/binnie/botany/genetics/EnumFlowerType.java

Line 50 in 4946c9d

FUCHSIA,

Foxglove - two.

Binnie/botany/src/main/java/binnie/botany/genetics/EnumFlowerType.java

Line 37 in 4946c9d

FOXGLOVE(2),

[RusTit]

From Idea:
Reports a null check followed by an instanceof check or a method call which will surely return false when null is passed (e.g. Class.isInstance). Since the instanceof operator always returns false for null, there is no need to have an additional null check.
Here is an example of a violation:
if (x != null && x instanceof String) { ... }
The quickfix changes this code to:
if (x instanceof String) { ... }

[RusTit]

@Nedelosk I will be grateful if you review this and #504 today

[mezz]

I think this is probably overkill, I'd be surprised if the JVM does not have some kind of optimization for the empty string.

[RusTit]

Hm. Probably you are right.
But now I used StringUtils.Empty (as you can see in #504)

[mezz]

Ok, sounds good.