patchGNU Octave - Patches: patch #8094, Octave-Forge: X-input grab screen...

 
 

patch #8094: Octave-Forge: X-input grab screen coordinates

Submitted by:  Juan Pablo Carbajal <juanpi>
Submitted on:  Tue 25 Jun 2013 04:18:57 PM UTC  
 
Category: NonePriority: 5 - Normal
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Thu 27 Jun 2013 07:58:22 AM UTC, comment #4:

Hello guys

I had a look at the grab.cc in the windows package. It is supposed to
do the same thing as the old grab.cc (same original author Andy
Adler), but only on Win* systems. The source code of grab.cc in the
windows package is windows specific.
Thus at leas a name confict with the two files.

The "old" grab.cc is XWindows specific and needs to be compilded with
the correct flags (see source code).
Proposition: rename the grab.cc in the msicellanous package to
Xgrab.cc and make the monor change in xinput.m

I hope this helps

Andreas

Andreas Stahel <sha1>
Wed 26 Jun 2013 07:20:16 PM UTC, comment #3:

Let's add Michael Goffioul (windows package maintainer) to the cc list.
I only noted that there's another grab.cc; I wouldn't know what to do next.
Michael might have some useful suggestions.

FYI, I compared the two grab.cc files and AFAICS they're mostly identical save for some lay-out and minor details.
So I'd say Andrea Stahel probably got it wrong - grab.cc does live on in an octave-forge package.

Philip Nienhuis <philipnienhuis>
Project Member
Wed 26 Jun 2013 12:02:08 PM UTC, comment #2:

Thanks for your input Philip,

Some questions:
1. The windows package is for windows while the grab function work on X11. When will there be clashes? Shall we change the name of this grab to xgrab?

2. This function (according to Andrea Stahel who provide it) was in Octave and was removed, but never put in a package. Was there a particular reason for that?

Juan Pablo Carbajal <juanpi>
Tue 25 Jun 2013 09:08:45 PM UTC, comment #1:

There's already a grab.cc in the windows package which AFAICS does a similar thing.
So at least expect a name clash.

Note that Andy Adler (author of your grab.cc) is the original author of the windows package.

Philip Nienhuis <philipnienhuis>
Project Member
Tue 25 Jun 2013 04:18:57 PM UTC, original submission:

This functions would be a nice addition to a forge package, probably miscellaneous.

It was in Octave long ago but was removed due to ML incompatibility and system wm dependence.

juanpi will take care of it

Juan Pablo Carbajal <juanpi>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #28410:  grab.cc added by juanpi (5kB - text/x-c++src)
file #28411:  xinput.m added by juanpi (2kB - text/x-matlab)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by sha1 (Posted a comment)
  • -unavailable- added by philipnienhuis
  • -unavailable- added by juanpi (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 26 Jun 2013 07:20:16 PM UTCphilipnienhuisCarbon-Copy-=>Added -unavailable-
    Tue 25 Jun 2013 04:18:57 PM UTCjuanpiAttached File-=>Added grab.cc, #28410
      Attached File-=>Added xinput.m, #28411

    Back to the top


    Powered by Savane 3.1-cleanup