`UnitConstants.day` conflicts with IntelliJ's auto organize imports and 'd' suffix for doubles

[EotT123]

When using IntelliJ IDEA with auto organize imports enabled, a conflict occurs when referencing manifold.science.util.UnitConstants.day. IntelliJ interprets day incorrectly due to the 'd' suffix used for double literals.

import static manifold.science.util.UnitConstants.*;

Time time = 1.2day;

The code is parsed as 1.2d ay, where 'd' is incorrectly interpreted as the double suffix, and ay is an unrecognized symbol.
IntelliJ removes the static import of day, as it is unused.
This results in a compilation error: Cannot resolve symbol 'ay'

The same issue also occurs with manifold.science.measures.TimeUnit.Day;

(This behavior is also visible in GitHub's syntax highlighting.)

[rsmckinney]

Yeah, I've been aware of this for many moons. It's a general issue with Java's literal suffixes: d, f, l. You can just add a space between the expression and the unit:

Time time = 1.2 day;

But, I'm sure I can make the parser ignore the suffix in this case. Thanks for reporting this!

[EotT123]

The problem is that the automatic formatter removes the space (and the import afterwards, because it's considered an unused import.). I don't think I can disable this without disabling the formatter completely (or I haven't found that specific option yet)

[rsmckinney]

Ah, gotcha. Thanks