Spray tool pressure sensitivity broken?

[Lucian]

I just acquired a Wacom Bamboo CTL-471 to use with Inkscape. Thankfully I've had few problems getting it to work. The Calligraphy tool and the tweak tool work with pressure sensitivity as expect ( a joy to use!). But I've noticed that the Spray tool pays no attention to the "Use pressure of input device..." setting. The amount sprayed is determined by the value in the "Amount" box, and nothing else. Pressure on the pen makes no difference at all that I can see. Given that the difference is very clear with the other tools, I'm guessing there is a bug with the Spray tool implementation somehow.

I'm running Inkscape 0.91 on Mac OS 10.10.3.

[su_v (suv-lp)]

Reproduced with Inkscape 0.48.5, 0.91 r13725 and 0.91+devel r14200 on OS X 10.7.5, using a Wacom Bamboo CTH-470:
There is no noticeable effect of pen pressure on the amount of the sprayed objects (copy or clone mode does not make a difference).

Pressure sensitivity with this combination (Inkscape on OS X under X11) otherwise works ok (e.g. with calligraphy tool).

[jazzynico (jazzynico)]

The Spray tool share some code (including the pressure part) with the Tweak code. Could you please test the Tweak tool and confirm the pressure work or not?

[jazzynico (jazzynico)]

Also reproduced on Crunchbang Waldorf, Inkscape trunk rev.14204, with a Bamboo fun CTE 450 tablet.
Pressure does work as expected with the Calligraphy and Tweak tools, but not with the Spray.
I wonder if the pressure option could be linked to a parameter which is no longer used (the Spray tool was originally designed to support some additional features).
Investigations in progress.

[jazzynico (jazzynico)]

> I wonder if the pressure option could be linked to a parameter which is no longer used

Confirmed. The pressure is linked to a Force parameter which is not used for spraying. I suggest we link it to the population parameter (Amount in the UI) instead.

[jazzynico (jazzynico)]

The attached patch links the tablet pressure to the Amount setting and removes some unused code in the spray tool.
Tested on Crunchbang Waldorf with Inkscape trunk rev. 14205.

Tests and comments are welcome!

[jazzynico (jazzynico)]

Patch committed trunk rev. 14210.

If nobody complains about the link between pressure and the Amount setting, the patch can also be backported to the 0.91.x branch.

[su_v (suv-lp)]

Fix backported to 0.91.x in rev 13800.