-
Notifications
You must be signed in to change notification settings - Fork 777
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
Introduce cmp_tree test function #3510
Conversation
testcases/t/301-tree.t
Outdated
# • http://onyxneon.com/books/modern_perl/modern_perl_a4.pdf | ||
# (unless you are already familiar with Perl) | ||
# | ||
# Contains various tests that use the cmp_tree subroutine. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure how we should split the tests. The sanity_check
s are useful to test cmp_tree
itself but the rest of the tests could be moved to the files that test the specific function, eg 291-swap.t
.
It would also be possible to rewrite lots of the old tests to use the new function but I think that it would be counter-productive.
|
||
=item 1. | ||
|
||
Upper case letters H, V, S, T mean horizontal, vertical, stacked and tabbed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: maybe we can also add a floating layer eg with 'F'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But we can do this in a separate PR
Using the wrong X11 connection breaks the libev event handling model: xcb_flush() must be called immediately before handing control to libev. Before this fix: 1. xcb_prepare_cb would read and flush conn 2. restore_xcb_prepare_cb would read and flush restore_conn, BUT also inadvertantly call xcb_flush(conn), resulting in new events being filled into the XCB event queue 3. libev waits for new events 4. after 1 minute, libev times out and the events are processed Diagnosed using strace on testcases/complete-run.pl. related to commit 0d8b671 related to i3#3510
c7ba93c
to
3402470
Compare
Related to i3#3503
@Airblader how would you like to continue with this PR here? |
@orestisf1993 LGTM. Anything from your side? If not, feel free to merge it! |
Ok, I am merging it and we'll see if it needs improvements later. |
Related to #3503