After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 736944 - videoscale: vs_image_scale_4tap_Y offset should use stride to calculate buffer offset
videoscale: vs_image_scale_4tap_Y offset should use stride to calculate buffe...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
1.x
Other Linux
: Normal minor
: 1.4.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-09-19 07:06 UTC by Sanjay NM
Modified: 2014-09-24 07:12 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
videoscale: vs_image_scale_4tap_Y using stride to calculate offset instead of width (1.28 KB, patch)
2014-09-19 07:06 UTC, Sanjay NM 		committed Details | Review

Description Sanjay NM 2014-09-19 07:06:46 UTC 
Created attachment 286577 [details] [review]
videoscale: vs_image_scale_4tap_Y using stride to calculate offset instead of width

vs_image_scale_4tap_Y uses width as of now. Though width and stride will contain same value in case of Y data, it will be logical to do the offset calculation using stride rather than width
Comment 1 Sebastian Dröge (slomo) 2014-09-23 20:09:49 UTC 
Thanks for the patch. Please include the Bugzilla URL in commit messages as the last line in the future.


commit db296c592475cb362c4ebff16dea3436dd7d17aa
Author: Sanjay NM <sanjay.nm@samsung.com>
Date:   Fri Sep 19 12:31:49 2014 +0530

    videoscale: Use width instead of stride in buffer offset calculation
    
    https://bugzilla.gnome.org/show_bug.cgi?id=736944
Comment 2 Sebastian Dröge (slomo) 2014-09-23 20:12:44 UTC 
commit 5adff2e65aa999b9471a00465f37b1edc9bb7900
Author: Sebastian Dröge <sebastian@centricular.com>
Date:   Tue Sep 23 23:12:19 2014 +0300

    videoscale Use stride instead of width in more places