It's okay if you paint the code red though. That really does make it go faster. Stripes are dope too.
On Thu, Feb 23, 2017 at 10:27:36PM +0000, Mário Luzeiro wrote: > I am *sorry* > > Mario > > ________________________________________ > From: Chris Pavlina [pavlina.ch...@gmail.com] > Sent: 23 February 2017 22:06 > To: Mário Luzeiro > Cc: Simon Richter; KiCad Developers > Subject: Re: [Kicad-developers] Optimization flags in 3d-viewer > > On Thu, Feb 23, 2017 at 03:48:39PM +0000, Mário Luzeiro wrote: > > Yes I added it, because for 3D we need *speeeeeed* :P > > Things like this are speed holes. https://youtu.be/whnms4CLJys?t=42 > > *Please* do not add speed holes. If your code is not faster, add > optimizations you understand, and remove them after testing if they do > not show a marked improvement. This just makes the build messy... > > > > > What kind of warnings it is causing? O_o > > > > Mario > > ________________________________________ > > From: Kicad-developers > > [kicad-developers-bounces+mrluzeiro=ua...@lists.launchpad.net] on behalf of > > Simon Richter [simon.rich...@hogyros.de] > > Sent: 23 February 2017 15:25 > > To: KiCad Developers > > Subject: [Kicad-developers] Optimization flags in 3d-viewer > > > > Hi, > > > > in 3d-viewer/CMakeLists.txt we have > > > > set( CMAKE_C_FLAGS_RELEASE "-O3" ) > > set( CMAKE_CXX_FLAGS_RELEASE "-O3" ) > > > > Is that intentional? > > > > - it seems to be unused in the Make based builds > > - it overwrites any existing flags in these variables > > - it causes warnings on MSVC > > > > Simon > > > > > > _______________________________________________ > > Mailing list: https://launchpad.net/~kicad-developers > > Post to : kicad-developers@lists.launchpad.net > > Unsubscribe : https://launchpad.net/~kicad-developers > > More help : https://help.launchpad.net/ListHelp _______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : kicad-developers@lists.launchpad.net Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp
