← Back to team overview

kicad-developers team mailing list archive

Re: /include/xnode.h giving warnings

 

In line 121. I'm sorry, I had too little sleep last night.


________________________________
 From: Alexander Lunev <al.lunev@xxxxxxxxx>
To: KiCad Developers <kicad-developers@xxxxxxxxxxxxxxxxxxx> 
Sent: Thursday, June 13, 2013 12:46 PM
Subject: Re: [Kicad-developers] /include/xnode.h giving warnings
 


Yes, it's my fault.
Would somebody with a write access add 'return' in line 118 of xnode.h?

Best regards,
Alexander


________________________________
 From: Edwin van den Oetelaar <oetelaar.automatisering@xxxxxxxxx>
To: KiCad Developers <kicad-developers@xxxxxxxxxxxxxxxxxxx> 
Sent: Thursday, June 13, 2013 11:28 AM
Subject: [Kicad-developers] /include/xnode.h giving warnings
 

I am still on Wx 2.8 and building gave me lots of warnings about no
return() from non void function.
Possible fix here.

Greetings,
Edwin van den Oetelaar


bzr diff include/
=== modified file 'include/xnode.h'
--- include/xnode.h 2013-06-12 19:43:22 +0000
+++ include/xnode.h 2013-06-13 07:25:23 +0000
@@ -118,7 +118,7 @@
     }
     bool DeleteAttribute( const wxString& attrName )
     {
-        DeleteProperty( attrName );
+        return DeleteProperty( attrName );
     }
     wxXmlProperty* GetAttributes() const
     {

_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp



_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

References