kicad-developers team mailing list archive
Mailing list archive
indentation and code style was Re: Infinite look taking 100% cpu when building debug mode.
Raúl Sánchez Siles <rss@...>
Fri, 25 Apr 2008 13:53:34 +0200
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
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.
RaÃºl SÃ¡nchez Siles