# HG changeset patch # User Pádraig Brady <p...@draigbrady.com> # Date 1309255312 -3600 # Node ID 7948a86a7e9510db13baa318fe64e9015fdbbd72 # Parent 75575ce398615dff21e2309c6b97523e99499498 Dev: build: avoid GCC 4.6 warnings
diff -r 75575ce39861 -r 7948a86a7e95 crmd/election.c --- a/crmd/election.c Tue Jun 21 15:54:20 2011 +0200 +++ b/crmd/election.c Tue Jun 28 11:01:52 2011 +0100 @@ -445,6 +445,8 @@ } last_election_win = time(NULL); +#else + (void) last_election_win; #endif register_fsa_input(C_FSA_INTERNAL, I_ELECTION, NULL); g_hash_table_destroy(voted); diff -r 75575ce39861 -r 7948a86a7e95 crmd/lrm.c --- a/crmd/lrm.c Tue Jun 21 15:54:20 2011 +0200 +++ b/crmd/lrm.c Tue Jun 28 11:01:52 2011 +0100 @@ -621,6 +621,8 @@ if(non_empty) { crm_log_xml_debug(restart, "restart digest source"); } +#else + (void) non_empty; #endif free_xml(restart); @@ -1275,7 +1277,7 @@ fsa_cib_update(XML_CIB_TAG_STATUS, fragment, cib_quorum_override, rc, user_name); free_xml(fragment); - + (void) rc; } else if(safe_str_eq(crm_op, CRM_OP_LRM_QUERY)) { xmlNode *data = do_lrm_query(FALSE); xmlNode *reply = create_reply(input->msg, data); diff -r 75575ce39861 -r 7948a86a7e95 crmd/te_actions.c --- a/crmd/te_actions.c Tue Jun 21 15:54:20 2011 +0200 +++ b/crmd/te_actions.c Tue Jun 28 11:01:52 2011 +0100 @@ -240,7 +240,6 @@ lrm_op_t *op = NULL; xmlNode *state = NULL; xmlNode *rsc = NULL; - xmlNode *xml_op = NULL; xmlNode *action_rsc = NULL; enum cib_errors rc = cib_ok; @@ -310,7 +309,7 @@ op->user_data = generate_transition_key( transition_graph->id, action->id, target_rc, te_uuid); - xml_op = create_operation_update( + create_operation_update( rsc, op, CRM_FEATURE_SET, target_rc, __FUNCTION__, LOG_INFO); free_lrm_op(op); diff -r 75575ce39861 -r 7948a86a7e95 lib/cib/cib_ops.c --- a/lib/cib/cib_ops.c Tue Jun 21 15:54:20 2011 +0200 +++ b/lib/cib/cib_ops.c Tue Jun 28 11:01:52 2011 +0100 @@ -169,7 +169,6 @@ xmlNode *existing_cib, xmlNode **result_cib, xmlNode **answer) { const char *tag = NULL; - gboolean verbose = FALSE; enum cib_errors result = cib_ok; crm_debug_2("Processing \"%s\" event for section=%s", @@ -188,9 +187,6 @@ tag = crm_element_name(input); - if (options & cib_verbose) { - verbose = TRUE; - } if(safe_str_eq(XML_CIB_TAG_SECTION_ALL, section)) { section = NULL; diff -r 75575ce39861 -r 7948a86a7e95 lib/cib/cib_remote.c --- a/lib/cib/cib_remote.c Tue Jun 21 15:54:20 2011 +0200 +++ b/lib/cib/cib_remote.c Tue Jun 28 11:01:52 2011 +0100 @@ -364,20 +364,27 @@ if(private->passwd == NULL) { struct termios settings; - int rc; - - rc = tcgetattr (0, &settings); + int rtc; + + rtc = tcgetattr (0, &settings); settings.c_lflag &= ~ECHO; - rc = tcsetattr (0, TCSANOW, &settings); + if (rtc == 0) { + rtc = tcsetattr (0, TCSANOW, &settings); + } - fprintf(stderr, "Password: "); - crm_malloc0(private->passwd, 1024); - scanf("%s", private->passwd); - fprintf(stdout, "\n"); - /* fprintf(stderr, "entered: '%s'\n", buffer); */ + if (rtc) { + fprintf(stderr, "Error disabling terminal echo: [%d]\n", errno); + rc = cib_unknown; + } else { + fprintf(stderr, "Password: "); + crm_malloc0(private->passwd, 1024); + scanf("%s", private->passwd); + fprintf(stdout, "\n"); + /* fprintf(stderr, "entered: '%s'\n", buffer); */ - settings.c_lflag |= ECHO; - rc = tcsetattr (0, TCSANOW, &settings); + settings.c_lflag |= ECHO; + tcsetattr (0, TCSANOW, &settings); + } } if(private->server == NULL || private->user == NULL) { diff -r 75575ce39861 -r 7948a86a7e95 lib/common/utils.c --- a/lib/common/utils.c Tue Jun 21 15:54:20 2011 +0200 +++ b/lib/common/utils.c Tue Jun 28 11:01:52 2011 +0100 @@ -2253,7 +2253,7 @@ return TRUE; } - crm_info("Could not send update: %s=%s for %s", name, value, host?host:"localhost"); + crm_info("Could not send update: %s=%s for %s: %s", name, value, host?host:"localhost", reason); return FALSE; } diff -r 75575ce39861 -r 7948a86a7e95 lib/common/xml.c --- a/lib/common/xml.c Tue Jun 21 15:54:20 2011 +0200 +++ b/lib/common/xml.c Tue Jun 28 11:01:52 2011 +0100 @@ -1168,7 +1168,6 @@ const char *prefix, xmlNode *data, int depth, gboolean formatted) { xmlNode *a_child = NULL; - int child_result = 0; int offset = 0; int printed = 0; @@ -1230,7 +1229,7 @@ } for(a_child = __xml_first_child(data); a_child != NULL; a_child = __xml_next(a_child)) { - child_result = log_data_element( + log_data_element( log_level, file, function, line, prefix, a_child, depth+1, formatted); } diff -r 75575ce39861 -r 7948a86a7e95 pengine/clone.c --- a/pengine/clone.c Tue Jun 21 15:54:20 2011 +0200 +++ b/pengine/clone.c Tue Jun 28 11:01:52 2011 +0100 @@ -750,8 +750,6 @@ action_t *started = NULL; GListPtr gIter = rsc->children; - resource_t *last_start_rsc = NULL; - resource_t *last_stop_rsc = NULL; clone_variant_data_t *clone_data = NULL; get_clone_variant_data(clone_data, rsc); @@ -764,13 +762,6 @@ child_rsc->cmds->create_actions(child_rsc, data_set); clone_update_pseudo_status( child_rsc, &child_stopping, &child_starting, &child_active); - - if(is_set(child_rsc->flags, pe_rsc_starting)) { - last_start_rsc = child_rsc; - } - if(is_set(child_rsc->flags, pe_rsc_stopping)) { - last_stop_rsc = child_rsc; - } } /* start */ @@ -869,9 +860,7 @@ { node_t *node = NULL; GListPtr gIter = NULL; - clone_variant_data_t *clone_data = NULL; - get_clone_variant_data(clone_data, rsc); - + if(local_node == NULL) { crm_err("Can't colocate unrunnable child %s with %s", local_child->id, rsc->id); @@ -918,9 +907,7 @@ GListPtr gIter = NULL; GListPtr scratch = NULL; node_t *local_node = NULL; - clone_variant_data_t *clone_data = NULL; - get_clone_variant_data(clone_data, rsc); - + local_node = local_child->fns->location(local_child, NULL, current); if(local_node) { return find_compatible_child_by_node(local_child, local_node, rsc, filter, current); diff -r 75575ce39861 -r 7948a86a7e95 pengine/group.c --- a/pengine/group.c Tue Jun 21 15:54:20 2011 +0200 +++ b/pengine/group.c Tue Jun 28 11:01:52 2011 +0100 @@ -437,8 +437,6 @@ void group_expand(resource_t *rsc, pe_working_set_t *data_set) { GListPtr gIter = rsc->children; - group_variant_data_t *group_data = NULL; - get_group_variant_data(group_data, rsc); crm_debug_3("Processing actions from %s", rsc->id); diff -r 75575ce39861 -r 7948a86a7e95 pengine/main.c --- a/pengine/main.c Tue Jun 21 15:54:20 2011 +0200 +++ b/pengine/main.c Tue Jun 28 11:01:52 2011 +0100 @@ -99,7 +99,6 @@ { int flag; int argerr = 0; - gboolean allow_cores = TRUE; IPC_Channel *old_instance = NULL; crm_system_name = CRM_SYSTEM_PENGINE; @@ -114,7 +113,7 @@ usage(crm_system_name, LSB_EXIT_OK); break; case 'c': - allow_cores = TRUE; + /* ignore */ break; default: ++argerr; diff -r 75575ce39861 -r 7948a86a7e95 pengine/master.c --- a/pengine/master.c Tue Jun 21 15:54:20 2011 +0200 +++ b/pengine/master.c Tue Jun 28 11:01:52 2011 +0100 @@ -862,9 +862,7 @@ resource_t *rsc_lh, resource_t *rsc_rh, rsc_colocation_t *constraint) { GListPtr gIter = NULL; - clone_variant_data_t *clone_data = NULL; - get_clone_variant_data(clone_data, rsc_rh); - + CRM_CHECK(rsc_rh != NULL, return); if(is_set(rsc_rh->flags, pe_rsc_provisional)) { return; _______________________________________________ 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