Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes and features for cmd_behave #213

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

J0J0
Copy link

@J0J0 J0J0 commented Jun 3, 2018

This series of commits fixes

  • the broken focus/blur events for the behave command
  • a memory issue with the behave command

(see commits for details) as well as introduces the following new features:

  • more (useful) events for behave
  • possibility to return from the behave command.

In case the features are not wanted, i also have a "bugfix only" branch at behave_fixes.

J0J0 added 5 commits April 6, 2018 09:42
The effective situation was: focus and blur were synomnyms
and both got fired by both window events. This commit fixes
that bug.
The previous code would abort at runtime on constructs like
  xdotool selectwindow behave "%1" mouse-enter getactivewindow
because context->windows gets freed (e.g. in window_save) ...
Also convert some messages to debug output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant