resurrecting-open-source-projects / scrot

SCReenshOT - command line screen capture utility
Other
495 stars 49 forks source link

fix multiple bugs in scrot's -C option #242

Closed guijan closed 1 year ago

guijan commented 1 year ago

I've carefully read the Xlib manuals and some more, but this PR should be reviewed by someone more familiar with X programming.

N-R-K commented 1 year ago

Unfortunately I cannot test this since the -k options seems to require a compositor (and I don't want to go though the hassle of setting one up).

Can you confirm that you did some before/after testing (just in case) @guijan ? If yes, then I'll go ahead and merge this since the code looks OK to me.

guijan commented 1 year ago

Yeah, I've tested it. Here's a screenshot I just took. 2023-03-25-073826_956x2144_scrot

N-R-K commented 1 year ago

Thanks.