URL:
  <http://gna.org/patch/?3603>

                 Summary: CodingStyle: Do not initialize variables with
nonsensical values
                 Project: Freeciv
            Submitted by: cazfi
            Submitted on: Sat 19 Jan 2013 12:15:22 PM EET
                Category: docs
                Priority: 5 - Normal
                  Status: Ready For Test
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 2.3.4, 2.4.0, 2.5.0

    _______________________________________________________

Details:

This one goes against what earlier days was taught as good programming
practice, but with modern tools I think that *avoiding* variable
initialization "just in case" makes better code in the long run. Using wrong
value (with what variable happened to be initialized with) doesn't make it any
better than using uninitialized value, but such a initialization hides the
problem from debugging tools (gcc warnings, valgrind reports)

So, attached patch against CodingStyle says that variables for which there's
no correct initialization value, must be left uninitialized.



    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Sat 19 Jan 2013 12:15:22 PM EET  Name: DontInitialize.patch  Size: 759B 
 By: cazfi

<http://gna.org/patch/download.php?file_id=16973>

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?3603>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to