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.

Attachment: redundant_if_sentence.patch
Description: 205491641-redundant_if_sentence.patch

_______________________________________________
Pacemaker mailing list: Pacemaker@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

Reply via email to