On Sat, Sep 22, 2018 at 3:25 AM Michael Niedermayer <[email protected]> wrote: > > On Thu, Sep 20, 2018 at 10:39:37AM +0800, [email protected] wrote: > > On Wed, Sep 19, 2018 at 7:07 PM Michael Niedermayer > > <[email protected]> wrote: > > > > > > On Tue, Sep 18, 2018 at 09:37:29PM +0800, Jun Zhao wrote: > > > > Fixes ticket #7441. > > > > > > > > Signed-off-by: Jun Zhao <[email protected]> > > > > --- > > > > libavfilter/vf_deshake.c | 12 +++++++----- > > > > 1 files changed, 7 insertions(+), 5 deletions(-) > > > > > > this doesnt look correct > > > > > > the blocks that will be used are not going out of array? so the contrast > > > calculated from them also should not > > > > > Do you mean the correct way is changing the pos calculated from: > > pos = (y - i) * stride + (x - j); to pos = (y + i) * stride + (x + j) ? > > Iam not the author of vf_deshake, but whatever aligns the checked blocks > correctly, should be the correct solution
Agree, so I submitted V2 patch for this issue, Thanks. > > [...] > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > > Modern terrorism, a quick summary: Need oil, start war with country that > has oil, kill hundread thousand in war. Let country fall into chaos, > be surprised about raise of fundamantalists. Drop more bombs, kill more > people, be surprised about them taking revenge and drop even more bombs > and strip your own citizens of their rights and freedoms. to be continued > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
