Blender Git Loki

Cleanup: replace verbose checks with read-only attributes

These attributes checked for any baked / baking
since this is a common test that was performed in layout code.

Also follow our naming convention - using an 'is_/has_' prefix
in this case since "cache_baked_data" reads as if it's used to access
the baked data.

Cleanup: use single quotes for enum values

Cleanup: remove unused variables

Cleanup: naming for BKE_fluid particle functions

Create/Destroy are more commonly paired terms in BLI/BKE API's.

Cleanup: use uint

Cleanup: remove redundant string initialization

Fixed sized strings are always initialized & this is not done
elsewhere before calling BLI_path_join.

Remove since it's not needed and makes it read as if the function
might not initialize the output argument.

Cleanup: use snake case variable & function names

Cleanup: conform header guards to Blender's style

Cleanup: split smoke drawing out into it's own file

gpu_draw.c had generic sounding utility functions which were specific
to smoke drawing.

Split into it's own file so the functionality is clearly separated.

Cleanup: redundant struct declarations

Cleanup: sort struct declarations

Cleanup: sort file lists

Cleanup: use BKE_fluid prefix for fluid API

PyAPI: disable threading hack when fluid is disabled

Cleanup: style, always use braces

Fix deprecated DNA define checks

Also remove dead code which checked for SPACE_TIME.

Cleanup: remove contributors, license begin/end & doxy file argument

This had already been removed for all source files,
recent patches re-introduced them.

Cleanup: compiler warnings

Cleanup: clang-format

Cleanup: remove redundant headers