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;
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
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
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:
>
>
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?
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
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:
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
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:
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
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
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 )
12 matches
Mail list logo