kicad-developers team mailing list archive

[Chris Pavlina <pavlina.chris@xxxxxxxxx>]

On Wed, Dec 06, 2017 at 11:38:21PM +0000, Wayne Stambaugh wrote:
> Simon,
> 
> I'm OK with most of these patches but since when is it invalid to
> initialize a c array in c++ (patch 4)?  Isn't c a subset of c++

No, no it is not.

> or is
> this just a fix to get kicad to compile with msvc?  If it's the latter,
> the coding policy is pretty clear about that.
> 
> Cheers,
> 
> Wayne
> 
> On 12/05/2017 09:42 PM, Simon Richter wrote:
> > Hi,
> > 
> > these are a few things that cause build errors on MSVC, where gcc accepts
> > invalid code. Most of them are obvious.
> > 
> > The pcb_test_window executable needs to be linked with /SUBSYSTEM:WINDOWS
> > on Windows because it is a GUI app, which uses different startup code than
> > a console application.
> > 
> > The Boost.Test issue is a double definition of several symbols because some
> > objects expect that they need to bring their own definition, while others
> > link the shared library instead. Either is fine, but mixing is not allowed
> > (but ELF linking silently overrides symbols from shared libraries).
> > 
> >    Simon
> > 
> > Simon Richter (6):
> >   Add missing include
> >   Avoid nonstandard variable length array
> >   Avoid initialization from non-constexpr
> >   Avoid C-style array member init (illegal in C++)
> >   Link pcb_test_window with /SUBSYSTEM:WINDOWS
> >   Pass -DBOOST_TEST_DYN_LINK to all parts of test
> > 
> >  common/common_plotDXF_functions.cpp                | 26 ++++++++++++----------
> >  common/geometry/shape_poly_set.cpp                 |  5 +++--
> >  eeschema/qa/CMakeLists.txt                         |  3 +++
> >  eeschema/qa/test_module.cpp                        |  1 -
> >  eeschema/sch_line.cpp                              | 21 +++++++++++------
> >  include/class_plotter.h                            |  1 -
> >  include/utf8.h                                     |  4 +++-
> >  qa/pcb_test_window/CMakeLists.txt                  |  2 +-
> >  .../test_polygon_triangulation.cpp                 |  6 ++---
> >  9 files changed, 41 insertions(+), 28 deletions(-)
> > 
> > 
> > 
> > _______________________________________________
> > Mailing list: https://launchpad.net/~kicad-developers
> > Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> > Unsubscribe : https://launchpad.net/~kicad-developers
> > More help   : https://help.launchpad.net/ListHelp
> > 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp