Re: [Kicad-developers] [PATCH] Fix clang warnings

2014-06-02 Thread Lorenzo Marcantonio
On Tue, Jun 03, 2014 at 08:18:52AM +0200, Heiko Rosemann wrote: > Probably the way to do this formally correct and without relying on > implicit integer-boolean conversion would be the following: > if( (ip = s1.Intersect( s_next )) != 0 ) Best of all MISRA-C recommends if (0 != stuff()) because

Re: [Kicad-developers] [PATCH] Fix clang warnings

2014-06-02 Thread Heiko Rosemann
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/03/2014 04:49 AM, tiger12506 wrote: > This is a guess (i'm not OP), but it's pretty common to shun > assignment within an if condition. The second line probably just > masks the warning from the compiler -- the check that triggers the > warning p

Re: [Kicad-developers] [PATCH] Fix clang warnings

2014-06-02 Thread Bernhard Stegmaier
Hi, it warns because it is not clear whether (a) you really mean the assignment (and want to check if the result is zero) or (b) you just forgot one “=“ and you meant “ip == s1…”. The additional brackets are proposed in the compiler warning to silence the warning in case of (a). Beside a lot o

Re: [Kicad-developers] [PATCH] Fix clang warnings

2014-06-02 Thread tiger12506
This is a guess (i'm not OP), but it's pretty common to shun assignment within an if condition. The second line probably just masks the warning from the compiler -- the check that triggers the warning probably isn't exhaustive enough to check within the parentheses for assignment. On 6/2/2014

Re: [Kicad-developers] [PATCH] Fix clang warnings

2014-06-02 Thread Tomasz Wlostowski
On 02.06.2014 19:52, Bernhard Stegmaier wrote: Hi, building with clang (at least on OSX) and -Wall is really noisy so that you can hardly see what’s going wrong. So, I decided to fix the warnings… attached are 3 patches I prepared (against rev. 4911): *** clang-warnings-1.diff: Everything that

Re: [Kicad-developers] [PATCH] Fix clang warnings

2014-06-02 Thread Bernhard Stegmaier
Of course x >=0 … or x < 0… :) Some of the unused variables really don’t seem to be used… e.g. in pcb_calculator classes, where I really guess no OSX dependencies and don’t see any #define around... Regards, Bernhard On 02.06.2014, at 20:04, Lorenzo Marcantonio wrote: > On Mon, Jun 02, 2014

Re: [Kicad-developers] [PATCH] Fix clang warnings

2014-06-02 Thread Lorenzo Marcantonio
On Mon, Jun 02, 2014 at 07:52:52PM +0200, Bernhard Stegmaier wrote: > Removes unneeded checks (e.g., a check “x > 0” for an "unsigned x” is pretty > obsolete). x >= 0, I hope :D As for unused variable probably it's only a Mac thing (or clang has a better flow analyzer than g++), I don't see any

[Kicad-developers] [PATCH] Fix clang warnings

2014-06-02 Thread Bernhard Stegmaier
Hi, building with clang (at least on OSX) and -Wall is really noisy so that you can hardly see what’s going wrong. So, I decided to fix the warnings… attached are 3 patches I prepared (against rev. 4911): *** clang-warnings-1.diff: Everything that probably should be fixed: * Disable “overloaded

Re: [Kicad-developers] Jenkins halted?

2014-06-02 Thread Marco Ciampa
On Mon, Jun 02, 2014 at 11:04:21PM +1200, Blair Bonnett wrote: > For anybody wanting to use GitHub, I also have a mirror at > https://github.com/blairbonnett-mirrors/kicad which is updated every 10 > minutes. Henner Zeller used it to fork from for his contributions a couple > of months back so I as

Re: [Kicad-developers] [Kicad-lib-committers] Silk screens over pads and naming

2014-06-02 Thread Lorenzo Marcantonio
On Mon, Jun 02, 2014 at 10:32:18AM -0400, Jean-Paul Louis wrote: > Guys, > > I am reading all this thread, and I am amazed that people suggest a silk > shape that will HAVE TO BE REMOVED by the PCB fabricator. The industry took a long time to discover this, too :D And it isn't a full standard ye

Re: [Kicad-developers] [Kicad-lib-committers] Silk screens over pads and naming

2014-06-02 Thread Jean-Paul Louis
Guys, I am reading all this thread, and I am amazed that people suggest a silk shape that will HAVE TO BE REMOVED by the PCB fabricator. This is adding cost to a PCB without any value added. The example shown by Lorenzo is a very good example. The yellow rectangle has zero value AND an extra co

Re: [Kicad-developers] Jenkins halted?

2014-06-02 Thread Blair Bonnett
On 2 June 2014 22:35, jp charras wrote: > Le 02/06/2014 12:15, Povilas Kanapickas a écrit : > > On 02/06/14 12:36, Povilas Kanapickas wrote: > >> On 29/05/14 16:15, Nick Østergaard wrote: > >>> Hello > >>> > >>> I am not sure who maintains the Jenkins build bot at > >>> http://ci.kicad-pcb.org/,

Re: [Kicad-developers] Jenkins halted?

2014-06-02 Thread jp charras
Le 02/06/2014 12:15, Povilas Kanapickas a écrit : > On 02/06/14 12:36, Povilas Kanapickas wrote: >> On 29/05/14 16:15, Nick Østergaard wrote: >>> Hello >>> >>> I am not sure who maintains the Jenkins build bot at >>> http://ci.kicad-pcb.org/, but I have noticed that is has stopped. I am >>> not sur

Re: [Kicad-developers] Jenkins halted?

2014-06-02 Thread Povilas Kanapickas
On 02/06/14 12:36, Povilas Kanapickas wrote: On 29/05/14 16:15, Nick Østergaard wrote: Hello I am not sure who maintains the Jenkins build bot at http://ci.kicad-pcb.org/, but I have noticed that is has stopped. I am not sure who runs that, so I write to the list. Also the kicad-source-mirror

Re: [Kicad-developers] Jenkins halted?

2014-06-02 Thread Povilas Kanapickas
On 29/05/14 16:15, Nick Østergaard wrote: Hello I am not sure who maintains the Jenkins build bot at http://ci.kicad-pcb.org/, but I have noticed that is has stopped. I am not sure who runs that, so I write to the list. Also the kicad-source-mirror on github has stopped updating. Is it the same