Re: [Kicad-developers] STEP Export

2016-09-22 Thread Cirilo Bernardo
On Fri, Sep 23, 2016 at 2:09 AM, Simon Wells wrote: > Hey Cirilo, > > is there any chance of being able to get output/log either written out > to file or in the dialog (or another dialog) like how it is in the > netlist read dialog? > > I'll be working on that this weekend;

Re: [Kicad-developers] export_vrml.cpp warnings

2016-09-22 Thread Cirilo Bernardo
Thanks, that's probably a bad C habit on my part. :) The most ancient of C++ specs were more lenient with magic int-enum casts. All the more reason to use "enum class" from now on; after all it was explicitly to address problems like this. Fix attached; the patch also removes the few instances

Re: [Kicad-developers] STEP Export

2016-09-22 Thread Cirilo Bernardo
On Fri, Sep 23, 2016 at 12:18 AM, Wayne Stambaugh wrote: > On 9/21/2016 9:17 PM, Cirilo Bernardo wrote: > > On Thu, Sep 22, 2016 at 10:07 AM, Wayne Stambaugh > wrote: > >> On 9/21/2016 7:27 PM, Cirilo Bernardo wrote: > >>> OK, I've updated the branch

Re: [Kicad-developers] STEP Export

2016-09-22 Thread Cirilo Bernardo
Oops .. I meant STEP AP210 for electronic assemblies rather than AP 310. At any rate, what the kicad2step tool creates at the moment is AP214; I don't believe OCE currently has support for AP210. - Cirilo On Fri, Sep 23, 2016 at 12:13 PM, Cirilo Bernardo wrote: > >

Re: [Kicad-developers] STEP Export

2016-09-22 Thread Simon Wells
Hey wayne, the commit broke my build... in kicad2step.cpp lines 61-81 have _( "BLAH BLAH") which errors out as not convertible from wxstring to char *. I have patched it with .mb_str() and was preparing a patch but i am not sure if this is the correct way, care to comment before i send a patch?

Re: [Kicad-developers] STEP Export

2016-09-22 Thread Wayne Stambaugh
Simon, Using .mb_str() is valid. Using static_cast is more c++ish. Take a look at the "Conversion to C string" section of the wxString documentation: On 9/22/2016 10:42 AM, Simon Wells wrote: > Hey wayne, > > the commit broke my build... > > in kicad2step.cpp > > lines 61-81 have _( "BLAH

Re: [Kicad-developers] STEP Export

2016-09-22 Thread Simon Wells
Hey Cirilo, is there any chance of being able to get output/log either written out to file or in the dialog (or another dialog) like how it is in the netlist read dialog? On Fri, Sep 23, 2016 at 4:04 AM, Simon Wells wrote: > Hmm weird, > > kicad2step:

Re: [Kicad-developers] STEP Export

2016-09-22 Thread Simon Wells
Hmm weird, kicad2step: /Volumes/simon/kicad-app/bin/kicad.app/kicad2step i think its to do with the embedding of the other application .apps, i have a dodgy fix but i will wait for comments from cirilo before providing a proper patch On Fri, Sep 23, 2016 at 3:44 AM, Simon Wells

Re: [Kicad-developers] [PATCH] Added explicit conversion from wxstring to cstring

2016-09-22 Thread Wayne Stambaugh
I pushed your patch to the product master branch. Thanks! On 9/22/2016 10:50 AM, Simon Wells wrote: > --- > utils/kicad2step/kicad2step.cpp | 18 +- > 1 file changed, 9 insertions(+), 9 deletions(-) > > > > ___ > Mailing list:

Re: [Kicad-developers] STEP Export

2016-09-22 Thread Bernhard Stegmaier
If I remember correctly this should only be enabled when some external tool is in the right place? Don’t know if this works yet on OSX? > On 22 Sep 2016, at 17:27, Simon Wells wrote: > > i am sure i have done something stupid but for me the export->STEP > option is greyed

Re: [Kicad-developers] STEP Export

2016-09-22 Thread Simon Wells
yeah apparently so, i am just trying to figure out where its looking. Which might be /Applications instead of inside the bundle On Fri, Sep 23, 2016 at 3:37 AM, Bernhard Stegmaier wrote: > If I remember correctly this should only be enabled when some external tool > is

[Kicad-developers] export_vrml.cpp warnings

2016-09-22 Thread Simon Wells
i am getting the following warning /Users/simon/kicad-app/kicad/pcbnew/exporters/export_vrml.cpp:1624:18: warning: comparison of constant -1 with expression of type 'VRML_COLOR_INDEX' is always false [-Wtautological-constant-out-of-range-compare] if( colorIdx == -1 )