Author: sveinung Date: Mon Jul 4 16:42:13 2016 New Revision: 33161 URL: http://svn.gna.org/viewcvs/freeciv?rev=33161&view=rev Log: An action probability check shouldn't change it.
Declare it const. See patch #7366 Modified: trunk/common/actions.c trunk/common/actions.h Modified: trunk/common/actions.c URL: http://svn.gna.org/viewcvs/freeciv/trunk/common/actions.c?rev=33161&r1=33160&r2=33161&view=diff ============================================================================== --- trunk/common/actions.c (original) +++ trunk/common/actions.c Mon Jul 4 16:42:13 2016 @@ -62,10 +62,14 @@ const struct output_type *target_output, const struct specialist *target_specialist); -static inline bool action_prob_is_signal(action_probability probability); -static inline bool action_prob_not_relevant(action_probability probability); -static inline bool action_prob_unknown(action_probability probability); -static inline bool action_prob_not_impl(action_probability probability); +static inline bool +action_prob_is_signal(const action_probability probability); +static inline bool +action_prob_not_relevant(const action_probability probability); +static inline bool +action_prob_unknown(const action_probability probability); +static inline bool +action_prob_not_impl(const action_probability probability); /************************************************************************** Initialize the actions and the action enablers. @@ -2754,7 +2758,7 @@ Returns TRUE iff the given action probability belongs to an action that may be possible. **************************************************************************/ -bool action_prob_possible(action_probability probability) +bool action_prob_possible(const action_probability probability) { return ACTPROB_IMPOSSIBLE != probability && ACTPROB_NA != probability; } @@ -2763,7 +2767,8 @@ Returns TRUE iff the given action probability represents the lack of an action probability. **************************************************************************/ -static inline bool action_prob_not_relevant(action_probability probability) +static inline bool +action_prob_not_relevant(const action_probability probability) { return ACTPROB_NA == probability; } @@ -2772,7 +2777,8 @@ Returns TRUE iff the given action probability represents that support for finding this action probability currently is missing from Freeciv. **************************************************************************/ -static inline bool action_prob_not_impl(action_probability probability) +static inline bool +action_prob_not_impl(const action_probability probability) { return ACTPROB_NOT_IMPLEMENTED == probability; } @@ -2785,7 +2791,8 @@ state the player don't have access to. It may be possible for the player to later gain access to this game state. **************************************************************************/ -static inline bool action_prob_unknown(action_probability probability) +static inline bool +action_prob_unknown(const action_probability probability) { return ACTPROB_NOT_KNOWN == probability; } @@ -2794,7 +2801,8 @@ Returns TRUE iff the given action probability represents a special signal value rather than a regular action probability value. **************************************************************************/ -static inline bool action_prob_is_signal(action_probability probability) +static inline bool +action_prob_is_signal(const action_probability probability) { return probability < 0 || probability > 200; } Modified: trunk/common/actions.h URL: http://svn.gna.org/viewcvs/freeciv/trunk/common/actions.h?rev=33161&r1=33160&r2=33161&view=diff ============================================================================== --- trunk/common/actions.h (original) +++ trunk/common/actions.h Mon Jul 4 16:42:13 2016 @@ -373,7 +373,7 @@ action_probability action_prob_self(const struct unit *actor, const int action_id); -bool action_prob_possible(action_probability probability); +bool action_prob_possible(const action_probability probability); action_probability action_prob_new_impossible(void); action_probability action_prob_new_not_relevant(void); _______________________________________________ Freeciv-commits mailing list Freeciv-commits@gna.org https://mail.gna.org/listinfo/freeciv-commits