Madi and Vasek,
I have not been able to look at the diff, and Martin Landa should have a chance to look this over in detail too and comment because he is doing most of the GUI code work currently.
The current mouse API is used in many display contexts in GRASS currently--main display, georectifer display, profile display, digitizing display, etc.
If this is an improvement to the existing API, it might be a good addition to GRASS. However, I'm reluctant to add a *new* mouse API in *addition* to the existing API because that would make the code much more difficult to use and understand for future coders and maintainers.
Michael
______________________________
C. Michael Barton
Director, Center for Social Dynamics & Complexity
Professor of Anthropology, School of Human Evolution & Social Change
Arizona State University
Tempe, AZ 85287-2402
USA
voice: 480-965-6262 (SHESC), 480-727-9746 (CSDC)
fax: 480-965-7671(SHESC), 480-727-0709 (CSDC)
www: http://csdc.asu.edu, http://shesc.asu.edu
http://www.public.asu.edu/~cmbarton
On Aug 18, 2011, at 7:57 PM, grass-dev-request@lists.osgeo.org wrote:
Date: Wed, 17 Aug 2011 17:42:43 +0200
From: Margherita Di Leo <dileomargherita@gmail.com>
Subject: [GRASS-dev] Re: wxGUI: MapDisplay mouse events
To: grass-dev@lists.osgeo.org
Message-ID:
<CA+V6ZhWvNYwQrnUg5akTRm=AVc4BhEzX7sap7Gy7z5u_y8aOkQ@mail.gmail.com>
Content-Type: text/plain; charset="utf-8"Hi Vasek,
I just tried your API in my rstream GUI and it works very well, thanks a lot
for your work! To devs who have SVN writing permission, may I ask if is it
possible to commit this patch into the core code please?Thanks,
madi