Fix for #35373, calling any paint operator from operator search menu
authorAntony Riakiotakis <kalast@gmail.com>
Sat, 18 May 2013 15:24:25 +0000 (15:24 +0000)
committerAntony Riakiotakis <kalast@gmail.com>
Sat, 18 May 2013 15:24:25 +0000 (15:24 +0000)
commit1c314aae73342995bf47622bc48b518f8d71b402
treeaea474976be5b50ea58e76aa2190c9c84bb1ab22
parenteaf493d323ce778c0b68a793b9cd6c813315aea3
Fix for #35373, calling any paint operator from operator search menu
with mouse crashes blender.

Issue here is that paint operator invocation, immediately calls modal,
which checks if event is the same as initialization event plus keyup.
Since using the mouse on amenu calls the operator on mouse button up,
the operator ended immediately and that cause immediate cleanup and
crash,

Worked around this by forbidding the operator to finish on first modal
call (which is expected by all the asserts on invoke anyway). This
prevents the crash but requires an extra up event to end.

Also, ported part of Jason Wilkins' patch to stroke code to allow for
stroke sampling to be done before a dab is first performed and as soon
as the stroke begins
source/blender/editors/sculpt_paint/paint_stroke.c