Electronics & Programming

develissimo

Open Source electronics development and programming

  • You are not logged in.
  • Root
  • » KiCAD
  • » [Kicad-developers] USE_WX_ZOOM update. [RSS Feed]

#1 Jan. 28, 2011 17:47:26

Wayne S.
Registered: 2009-11-02
Reputation: +  0  -
Profile   Send e-mail  

[Kicad-developers] USE_WX_ZOOM update.


I am pretty much done removing USE_WX_ZOOM and all of the unused code
associated with it's removal. Normally I would just commit this code but as it
is such a large change with potential for rendering issues, I want to get some
input before I commit the changes.

Here is the short list of the changes:

* All occurrences if #ifdef USE_WX_ZOOM are gone as well as any associated code
within the #else/#endif block ( old zoom code ).

* Removed the build option for USE_WX_ZOOM from CMakeList.txt.

* All scaling code in base screen object removed.

* Fixed buffered paint and buffered client DC on Windows. Buffering works
properly on Linux and windows. Needs tested on OSX.

* Modified kicad_device_context.h to automatically use buffering on platforms
where double buffering is supported natively.

* All of the scaled versions of the drawing functions in gr_basic.cpp are gone
as well as any support code.

* ActiveScreen global variable completely removed from eeschema and gerbview.

* The number of function calls has been cut at least in half by removing the
scaling in gr_basic.cpp which should yield a minor performance improvement.

Currently buffering cannot be used to draw in the client area except when
redrawing the entire visible area. This due to the way objects such as lines
and arcs are drawn during editing by XORing over top of the current item and
redrawing it in the new position without redrawing the background. I have
implemented this by using the current INSTALL_DC in the Recadre_Trace()
(renamed RedrawScreen )method in zoom.cpp and created an unbuffered version of
the macro INSTALL_UNBUFFERED_DC for drawing everywhere else.

There may be some corner cases where the drawing doesn't always work properly
that I missed. Is there any reason why I shouldn't commit this code? If there
are objections, I could put my repo on launchpad or create a patch so that
everyone can take a look at it.

Let me know what you think.

Wayne

_______________________________________________
Mailing list:https://launchpad.net/~kicad-developersPost to : kicad-developers@lists.launchpad.net
Unsubscribe :https://launchpad.net/~kicad-developersMore help :https://help.launchpad.net/ListHelp

Offline

#2 Jan. 28, 2011 20:07:51

Marco S.
Registered: 2009-11-02
Reputation: +  0  -
Profile   Send e-mail  

[Kicad-developers] USE_WX_ZOOM update.


On 28/gen/2011, at 18.46, Wayne Stambaugh wrote:

> I am pretty much done removing USE_WX_ZOOM and all of the unused code

Is ok for me :)

--
Marco

_______________________________________________
Mailing list:https://launchpad.net/~kicad-developersPost to : kicad-developers@lists.launchpad.net
Unsubscribe :https://launchpad.net/~kicad-developersMore help :https://help.launchpad.net/ListHelp

Offline

#3 Jan. 28, 2011 20:27:15

jean-pierre c.
Registered: 2009-11-02
Reputation: +  0  -
Profile   Send e-mail  

[Kicad-developers] USE_WX_ZOOM update.


Le 28/01/2011 18:46, Wayne Stambaugh a écrit :I am pretty much done removing USE_WX_ZOOM and all of the unused code
associated with it's removal. Normally I would just commit this code but as it
is such a large change with potential for rendering issues, I want to get some
input before I commit the changes.There may be some corner cases where the drawing doesn't always work properly
that I missed. Is there any reason why I shouldn't commit this code? If there
are objections, I could put my repo on launchpad or create a patch so that
everyone can take a look at it.

Let me know what you think.

WayneThis is OK for me.

--
Jean-Pierre CHARRAS


_______________________________________________
Mailing list:https://launchpad.net/~kicad-developersPost to : kicad-developers@lists.launchpad.net
Unsubscribe :https://launchpad.net/~kicad-developersMore help :https://help.launchpad.net/ListHelp

Offline

  • Root
  • » KiCAD
  • » [Kicad-developers] USE_WX_ZOOM update. [RSS Feed]

Board footer

Moderator control

Enjoy the 24th of October
PoweredBy

The Forums are managed by develissimo stuff members, if you find any issues or misplaced content please help us to fix it. Thank you! Tell us via Contact Options
Leave a Message
Welcome to Develissimo Live Support