[FFmpeg-devel] gdigrab: fix HIDPI support

Carl Eugen Hoyos ceffmpeg at gmail.com
Sun Jan 27 02:45:18 EET 2019


2019-01-26 18:53 GMT+01:00, Dilshod Mukhtarov <dilshodm at gmail.com>:
> HI, this is the patch that fixes HIDPI support in gdigrab

> +    double h_dpr;   // Horizontal device pixel ratio
> +    double v_dpr;   // Vertical device pixel ratio

I would expect these to be AVRational, if this is not
possible, it should be explained why.

Please put "else" on the same line as "}", no linebreak
between "}" and "else".

> 1) Mouse position was not calculated properly in area or window mode
> 2) In window mode the size of window was not calculated properly (cropped)

This may not apply here, but typically, if a patch says "fixes A and B",
it should be split in two patches to ease review and future debugging.

Carl Eugen


More information about the ffmpeg-devel mailing list