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

                 Summary: Clean up the actions module
                 Project: Freeciv
            Submitted by: sveinung
            Submitted on: Sun 08 Sep 2013 11:28:47 PM GMT
                Category: general
                Priority: 5 - Normal
                  Status: Ready For Test
                 Privacy: Public
             Assigned to: sveinung
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 2.6.0

    _______________________________________________________

Details:

* Move evaluation of a single action enabler to a separate function
* Change name of boolean functions to start with is_ as the code style says

Style question: The function for evaluating a singe action enabler is local.
Is the policy to always prototype a local function at the top of the C file or
does "Local functions should always be declared as static" in the code style
also include the implicit declaration in the definition?



    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Sun 08 Sep 2013 11:28:47 PM GMT  Name: action_clean_up.patch  Size: 8kB 
 By: sveinung

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

    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  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