Blender Git Commits

Cleanup: Create function to duplicate stroke to avoid code duplication.

Cleanup: De-duplicate dynamic enum function used for setting GP brush icons

It turned out that there were slight differences between these two
versions (namely, where they searched for the list of active brushes),
but it's not too much of a stretch to use the same code for this with
a few minor tweaks, as per this commit.

Cleanup: Remove unused/deprecated dynamic enum stuff

New option to separate Points

Added an option to separate selected points and not only whole stroke.

Also some cleanup task as rename vars and reorganize code.

Add P key shortcut for Separate operator

UI changes for color picker

* Colors have labels, so previews don't need to be as big, GP_BOX_SIZE set to 32.
* More padding between items, they are too close to each other at the moment GP_BOX_GAP set to 24.
* Round boxes are a bit less round to match the rest of the UI 0.2f (before used 0.4f)
* Cursor should not change to color picker. We are performing an action (selecting an item), not picking UI colors per se.

Thanks Pablo Vazquez for the feedback

Merge branch 'blender2.8' into greasepencil-object

Cleanup: Define a shared poll method for edit 3D

Cleanup: Remove unnecessary flags and fix typo

New Separate operator

This operator works similar to mesh separate operator, but for grease pencil strokes.

Two options:

a) Selected Strokes
b) Active Layer

Move int <-> float 2d vector funcs from GP code to Math Lib

We now have the following two new functions:
* copy_v2fl_v2i(float r[2], const int a[2]) <-- Copies 2D int vector to float vector
* round_v2i_v2fl(int r[2], const float a[2]) <-- Copies 2D float vector to int vector, by rounding the values

Merge branch 'blender2.8' into greasepencil-object

Cleanup: Use standard mathlib functions here, instead of inlined pointer magic/etc.

More self-contained fix for previous commit

Cleanup: Various fixes

* Remove redundant operations (e.g. extra copies of data)

* Remove duplicate functions (e.g. vector copying, for non-floats)
TODO: We still have copy_v2int_v2float() and copy_v2float_v2int()
to fix (i.e. rename + move to BLI_math.h)

* Cleanup redundant struct forward defs in headers (including some duplicates)

* Hacky fix for compiler warning in gpencil_colorpick.c (about const vs non-const)
UI_FSTYLE_WIDGET is const, but UI_text_clip_middle_ex() needs non-const

Cleanup: More simple fixes

Cleanup: Style fixes

Cleanup: Move ED_gpencil_stroke_minmax() to blenkernel

The rest of the boundbox/minmax code is already there. We might as well
have the function for grabbing this info from a single stroke there too.

Cleanup: Reshuffling order of code in file

In future: Keep related code together, instead of just dumping
it "wherever" (aka at the end of the file, since it's most
convenient right now)

(In the process, I've ended up flagging a few pieces of code
for further investigation. They likely duplicate some code
elsewhere in other modules)

Cleanup: Prefix all exported functions with module prefix