kicad-developers team mailing list archive
Mailing list archive
Re: [PATCH 2/2] Remove redundant statement in WRL2BASE::ReadNode
Kevin Cozens <kevin@xxxxxxxxx>
Thu, 13 Apr 2017 21:58:02 -0400
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0
On 2017-04-13 06:06 PM, Cirilo Bernardo wrote:
On Thu, Apr 13, 2017 at 1:34 PM, Clemens Koller <cko@xxxxxxxxx> wrote:
These lines scream for some comments in the source...
I wouldn't get it, too.
What sort of comment: "this is really supposed to have two sequential
calls to the same function, so don't change it"? For me that makes no
sense. If anyone is going to play with parsers they should be familiar
with the standard that is being implemented;
How much hand-holding are we expected to do?
That duplicate call looks really suspicious at first glance. A one line
comment saying it is deliberate and not a mistake is all that is needed. Any
additional explanation can be left for elsewhere in the source code.
http://www.ve3syb.ca/ |"Nerds make the shiny things that distract
Owner of Elecraft K2 #2172 | the mouth-breathers, and that's why we're
#include <disclaimer/favourite> | --Chris Hardwick