kicad-developers team mailing list archive

[Raúl Sánchez Siles <rss@...>]

El Jueves, 24 de Abril de 2008, Dick Hollenbeck escribió:
> Raúl Sánchez Siles wrote:
> > El Jueves, 24 de Abril de 2008, Raúl Sánchez Siles escribió:
> >> Hello Dick and thanks for you much clarifying anwer:
> >>
> >> I'm thinking of a solution which would involve using a wxProcess
> >> variable to redirect pipes into it or something like that. I'm doing
> >> some tests and preparing a patch for this, so I get this working you'll
> >> have news about it.
> >>
> >> If I'm right I think this isn't a problem only covering debug builds,
> >> but also general builds as long as the child process output enough
> >> information, so it's something where care should be taken.
> >>
> >> Thanks for the help, regards,
> >
> > Find attached a first try. Tested and works. I don't know yet the
> > possible drawbacks of this.
> >
> > What do you think about it?
>
> It does not fix the so called problem. So you go back to the drawing
> board, if it still bothers you.
>
> I just tried it and PCBNEW hung up when run from kicad.
>
>
> I frankly think its a waste of time, but just to provide a better
> playground for you, I am submitting a patch which consolidates the
> solution into one place in common.cpp
>
>
> Dick

Thanks a lot for the patch. That was the correct implementation for the 
patch I sent as proof of concept. In my case, pcbnew worked, but since you 
patch is more comprehensive I've tested it as well and looks working.

What I've noticed is the patch is somewhat noisy because changes in 
indentation style. Where tabs were used, you changed it for white spaces. I'm 
not sure guidelines are established for code style.

As you may have noticed, I don't have a deep knowledge of kicad code, butI 
think commits could be clearer if there would be a code style. If it is, I 
volunteer to send patches in order to comply with that style and maybe do 
some other minor changes like code documentation/comments translations.
  
Regards,

-- 
Raúl Sánchez Siles