Skip to content

Commit b95cef7

Browse files
committed
Refactor: libcrmcommon: Make vfields file-scope in patchset.c
Reduce some duplication Signed-off-by: Reid Wahl <[email protected]>
1 parent 8a72a9b commit b95cef7

File tree

1 file changed

+6
-23
lines changed

1 file changed

+6
-23
lines changed

lib/common/patchset.c

Lines changed: 6 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,12 @@
2626
#include <crm/common/xml_internal.h> // CRM_XML_LOG_BASE, etc.
2727
#include "crmcommon_private.h"
2828

29+
static const char *const vfields[] = {
30+
PCMK_XA_ADMIN_EPOCH,
31+
PCMK_XA_EPOCH,
32+
PCMK_XA_NUM_UPDATES,
33+
};
34+
2935
/* Add changes for specified XML to patchset.
3036
* For patchset format, refer to diff schema.
3137
*/
@@ -185,11 +191,6 @@ xml_create_patchset_v2(xmlNode *source, xmlNode *target)
185191
xmlNode *v = NULL;
186192
xmlNode *version = NULL;
187193
xmlNode *patchset = NULL;
188-
const char *vfields[] = {
189-
PCMK_XA_ADMIN_EPOCH,
190-
PCMK_XA_EPOCH,
191-
PCMK_XA_NUM_UPDATES,
192-
};
193194

194195
pcmk__assert(target != NULL);
195196

@@ -330,12 +331,6 @@ int
330331
pcmk__xml_patchset_versions(const xmlNode *patchset, int source[3],
331332
int target[3])
332333
{
333-
static const char *const vfields[] = {
334-
PCMK_XA_ADMIN_EPOCH,
335-
PCMK_XA_EPOCH,
336-
PCMK_XA_NUM_UPDATES,
337-
};
338-
339334
const xmlNode *version = pcmk__xe_first_child(patchset, PCMK_XE_VERSION,
340335
NULL, NULL);
341336
const xmlNode *source_xml = pcmk__xe_first_child(version, PCMK_XE_SOURCE,
@@ -389,12 +384,6 @@ xml_patch_version_check(const xmlNode *xml, const xmlNode *patchset)
389384
int del[] = { 0, 0, 0 };
390385
int rc = pcmk_rc_ok;
391386

392-
const char *vfields[] = {
393-
PCMK_XA_ADMIN_EPOCH,
394-
PCMK_XA_EPOCH,
395-
PCMK_XA_NUM_UPDATES,
396-
};
397-
398387
for (lpc = 0; lpc < PCMK__NELEM(vfields); lpc++) {
399388
crm_element_value_int(xml, vfields[lpc], &(this[lpc]));
400389
crm_trace("Got %d for this[%s]", this[lpc], vfields[lpc]);
@@ -928,12 +917,6 @@ pcmk__cib_element_in_patchset(const xmlNode *patchset, const char *element)
928917
bool
929918
xml_patch_versions(const xmlNode *patchset, int add[3], int del[3])
930919
{
931-
static const char *const vfields[] = {
932-
PCMK_XA_ADMIN_EPOCH,
933-
PCMK_XA_EPOCH,
934-
PCMK_XA_NUM_UPDATES,
935-
};
936-
937920
const xmlNode *version = pcmk__xe_first_child(patchset, PCMK_XE_VERSION,
938921
NULL, NULL);
939922
const xmlNode *source = pcmk__xe_first_child(version, PCMK_XE_SOURCE, NULL,

0 commit comments

Comments
 (0)