Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-14 Thread Matt Oliver
On 13 February 2016 at 05:33, Γιώργος Μεταξάκης wrote: > checked with patcheck also. > only "problem" is > "Missing changelog entry (ignore if minor change)' > LGTM, applied. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailm

Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-12 Thread Γιώργος Μεταξάκης
checked with patcheck also. only "problem" is "Missing changelog entry (ignore if minor change)' On Fri, Feb 12, 2016 at 11:51 AM Moritz Barsnick wrote: > On Fri, Feb 12, 2016 at 09:25:54 +0100, Hendrik Leppkes wrote: > > On Fri, Feb 12, 2016 at 9:21 AM, Γιώργος Μεταξάκης > wrote: > > > i'm ver

Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-12 Thread Moritz Barsnick
On Fri, Feb 12, 2016 at 09:25:54 +0100, Hendrik Leppkes wrote: > On Fri, Feb 12, 2016 at 9:21 AM, Γιώργος Μεταξάκης wrote: > > i'm very sorry for those mistakes > That looks much better already, unfortunately there is still tabs in the > patch. A good hint for new contributors is to use: ./tools

Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-12 Thread Γιώργος Μεταξάκης
didn't knew about that ticket. On Fri, Feb 12, 2016 at 10:57 AM Carl Eugen Hoyos wrote: > Γιώργος Μεταξάκης gmail.com> writes: > > > > > i'm sending you a new patch with only 6 changes. > > > > The problem was that when you where capturing the > > > > screen, the mouse wasn't on the correct loc

Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-12 Thread Carl Eugen Hoyos
Γιώργος Μεταξάκης gmail.com> writes: > > > i'm sending you a new patch with only 6 changes. > > > The problem was that when you where capturing the > > > screen, the mouse wasn't on the correct location. Does this fix ticket #5008? If yes, please mention it in the commit message. Thank you, Ca

Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-12 Thread Γιώργος Μεταξάκης
I have changed that. On Fri, Feb 12, 2016 at 10:25 AM Hendrik Leppkes wrote: > On Fri, Feb 12, 2016 at 9:21 AM, Γιώργος Μεταξάκης > wrote: > > i'm very sorry for those mistakes > > i'm sending you a new patch with only 6 changes. > > The problem was that when you where capturing the screen, the

Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-12 Thread Hendrik Leppkes
On Fri, Feb 12, 2016 at 9:21 AM, Γιώργος Μεταξάκης wrote: > i'm very sorry for those mistakes > i'm sending you a new patch with only 6 changes. > The problem was that when you where capturing the screen, the mouse wasn't > on the correct location. > That looks much better already, unfortunately

Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-12 Thread Γιώργος Μεταξάκης
i'm very sorry for those mistakes i'm sending you a new patch with only 6 changes. The problem was that when you where capturing the screen, the mouse wasn't on the correct location. On Fri, Feb 12, 2016 at 2:01 AM James Darnley wrote: > On 2016-02-11 23:19, Γιώργος Μεταξάκης wrote: > > Subject:

Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-11 Thread James Darnley
On 2016-02-11 23:19, Γιώργος Μεταξάκης wrote: > Subject: [PATCH] mouse dpi awareness > > --- > libavdevice/gdigrab.c | 28 +++- > 1 file changed, 15 insertions(+), 13 deletions(-) > > diff --git a/libavdevice/gdigrab.c b/libavdevice/gdigrab.c > index 4428a34..60f184e 1006

Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-11 Thread Hendrik Leppkes
On Thu, Feb 11, 2016 at 11:19 PM, Γιώργος Μεταξάκης wrote: > Hello I have tried to send that email with git send-email but failed, so i > will send the patch here, i hope it is okay. > > I am an engineer at Jumpshare (https://jumpshare.com) working on the native > file sharing app for Windows. We

[FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-11 Thread Γιώργος Μεταξάκης
Hello I have tried to send that email with git send-email but failed, so i will send the patch here, i hope it is okay. I am an engineer at Jumpshare (https://jumpshare.com) working on the native file sharing app for Windows. We were using gdigrab device for screen capture, we played around a bit