Created by: benfry

Both Processing.py and Processing-R rely on the old constant to check whether shift is down. Inside handleRun() is the code:

final boolean shift = (modifiers & InputEvent.SHIFT_MASK) != 0;

Sadly, in earlier releases we only set a boolean for whether shift was down. I changed to pass the modifiers for greater flexibility, but we've been burned for it (non-shift modifiers don't appear to be used anywhere), not expecting something as fundamental as these constants to be changed.

Definitions for the old constant for shift:

InputEvent.SHIFT_MASK = Event.SHIFT_MASK;
Event.SHIFT_MASK = 1 << 0;

Unfortunately, the new constant uses a different value:

InputEvent.SHIFT_DOWN_MASK = 1 << 6;

And confirmed that getModifiers() and getModifiersEx() cannot be used interchangeably:

void draw() {
}

void mousePressed() {
  java.awt.event.MouseEvent me = (java.awt.event.MouseEvent) mouseEvent.getNative();
  int modifiers = me.getModifiers();
  int modifiersEx = me.getModifiersEx();
  //boolean shiftOld = (modifiers & java.awt.Event.SHIFT_MASK) != 0;
  //println(shiftOld);
  println("mods and old shift = " + ((modifiers & java.awt.Event.SHIFT_MASK) != 0));
  println("mods and new shift = " + ((modifiers & java.awt.event.InputEvent.SHIFT_DOWN_MASK) != 0));
  println("exmods and old shift = " + ((modifiersEx & java.awt.Event.SHIFT_MASK) != 0));
  println("exmods and new shift = " + ((modifiersEx & java.awt.event.InputEvent.SHIFT_DOWN_MASK) != 0));
  println();
}

So we may need to stick with the old approach and hold our nose. Could swap out a default implementation that sets the old modifiers for backwards compatibility, but is overridden by default, but we're using an abstract class there (again, following best principles), so we might be mucking with method signatures and break things there too.

Uh oh!

Switch to getModifiersEx() in processing.app #67

Description

Metadata

Metadata

Assignees

Labels

Type

Fields

Projects

Milestone

Relationships

Development

Issue actions

Switch to getModifiersEx() in `processing.app` #67