[Pacemaker] [PATCH]A redundant if sentence.

Andrew Beekhof andrew at beekhof.net
Wed Aug 4 08:24:35 UTC 2010


Looks good, applying.
Thanks!

2010/8/4  <renayama19661014 at ybb.ne.jp>:
> Hi,
>
> It is the patch of a redundant if sentence for pengine.
>
> void
> unpack_operation(
>        action_t *action, xmlNode *xml_obj, pe_working_set_t* data_set)
> {
> (snip)
>        if(safe_str_eq(class, "stonith")) {
>                action->needs = rsc_req_nothing;
>                value = "nothing (fencing op)";
>
>        } else if(value == NULL && safe_str_neq(action->task, CRMD_ACTION_START)) {
> (snip)
>        } else if(data_set->no_quorum_policy == no_quorum_ignore
>            || safe_str_eq(class, "stonith")) { *** ---> A redundant if sentence.
>                action->needs = rsc_req_nothing;
>                value = "nothing (default)";
>
>        } else if(data_set->no_quorum_policy == no_quorum_freeze
>                  && is_set(data_set->flags, pe_flag_stonith_enabled)) {
> (snip)
>
> Best Regards,
> Hideo Yamauchi.
>
> _______________________________________________
> Pacemaker mailing list: Pacemaker at oss.clusterlabs.org
> http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>
> Project Home: http://www.clusterlabs.org
> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
> Bugs: http://developerbugs.linux-foundation.org/enter_bug.cgi?product=Pacemaker
>
>




More information about the Pacemaker mailing list