Check usage of context managers

Question

Check usage of context managers

heuer opened this issue 7 months ago · 2 comments

It's a minor issue but several classes provide a context manager where it makes no sense, in my opinion. For example: Where is the use case for a context manager in Output, or CursorManager, or Cursor…?

In my opinion all context managers which have no valid use case should be removed.

Answer 1 · 2024-04-07T11:49:09.000Z

OutputLayout is another candidate where the value of a context manager is questionable.

Answer 2 · 2024-05-07T06:41:12.000Z

@flacjacket, @jwijenbergh, maybe WLR 0.17 offers a chance to clean this up?

However, I don't think this issue is important enough to delay a release of WLR 0.17.