[Bug 777475] compositor : Issue in computing position when pixel-aspect-ratio is not 1/1

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=777475

--- Comment #2 from Baby octopus <jagadishkamathk at gmail.com> ---
Thank you for the review. Sorry if this sounds naive. But should not the
position be re-calculated based on stretched width and height i.e., the
position too should be computed as follows

actual_pos_x = pos_x * par_n / par_d;
actual_pos_y = pos_y * par_n / par_d;

And use actual_pos_x and actual_pos_y for all pixel based operations? 

The reasoning behind the change is that, a user may not be aware of
pixel-aspect-ratio which typically comes in stream header. But the user would
be more aware of video resolution. It could be difficult for the user to adjust
pos_x and pos_y based on incoming stream's par. If the incoming stream keeps
changing its par dynamically(sometimes seen during content to ad transition in
TS streams), this will make it even more difficult

Thanks

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.