2019-01-26 18:53 GMT+01:00, Dilshod Mukhtarov <dilsh...@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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to