diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2021-01-21 13:08:25 +1000 |
---|---|---|
committer | Peter Hutterer <peter.hutterer@who-t.net> | 2021-01-21 14:11:37 +1000 |
commit | 9510dedad875099c32993246188050ea73ab1a9f (patch) | |
tree | 2471fbf77a8fef9e3e57ac4cb757472d6cb96016 /keytypes.c | |
parent | ff6a19ed8898ee8c79b2b2a2f87806353d89f5d4 (diff) |
Replace WARN[1-9], ERROR[1-9], etc. with their unnumbered version
Those macros date back to when varargs weren't a thing but they've
been #defined to the same value for 17 years now.
Patch generated with:
for action in WARN INFO ERROR ACTION FATAL WSGO; do
sed -i "s/${action}[1-9]/${action}/g" `git ls-files`
done
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Diffstat (limited to 'keytypes.c')
-rw-r--r-- | keytypes.c | 88 |
1 files changed, 44 insertions, 44 deletions
@@ -274,7 +274,7 @@ FindMatchingKeyType(KeyTypesInfo * info, KeyTypeInfo * new) static Bool ReportTypeBadWidth(const char *type, int has, int needs) { - ERROR3("Key type \"%s\" has %d levels, must have %d\n", type, has, needs); + ERROR("Key type \"%s\" has %d levels, must have %d\n", type, has, needs); ACTION("Illegal type definition ignored\n"); return False; } @@ -326,7 +326,7 @@ AddKeyType(XkbDescPtr xkb, KeyTypesInfo * info, KeyTypeInfo * new) if (((old->defs.fileID == new->defs.fileID) && (warningLevel > 0)) || (warningLevel > 9)) { - WARN1("Multiple definitions of the %s key type\n", + WARN("Multiple definitions of the %s key type\n", XkbAtomGetString(NULL, new->name)); ACTION("Earlier definition ignored\n"); } @@ -342,7 +342,7 @@ AddKeyType(XkbDescPtr xkb, KeyTypesInfo * info, KeyTypeInfo * new) report = (old->defs.fileID == new->defs.fileID) && (warningLevel > 0); if (report) { - WARN1("Multiple definitions of the %s key type\n", + WARN("Multiple definitions of the %s key type\n", XkbAtomGetString(NULL, new->name)); ACTION("Later definition ignored\n"); } @@ -520,7 +520,7 @@ NextMapEntry(KeyTypeInfo * type) type->entries = uTypedCalloc(2, XkbKTMapEntryRec); if (type->entries == NULL) { - ERROR1("Couldn't allocate map entries for %s\n", TypeTxt(type)); + ERROR("Couldn't allocate map entries for %s\n", TypeTxt(type)); ACTION("Map entries lost\n"); return NULL; } @@ -535,7 +535,7 @@ NextMapEntry(KeyTypeInfo * type) XkbKTMapEntryRec); if (type->entries == NULL) { - ERROR1("Couldn't reallocate map entries for %s\n", TypeTxt(type)); + ERROR("Couldn't reallocate map entries for %s\n", TypeTxt(type)); ACTION("Map entries lost\n"); return NULL; } @@ -563,7 +563,7 @@ AddPreserve(XkbDescPtr xkb, { if (warningLevel > 9) { - WARN2("Identical definitions for preserve[%s] in %s\n", + WARN("Identical definitions for preserve[%s] in %s\n", PreserveIndexTxt(type, xkb, old), TypeTxt(type)); ACTION("Ignored\n"); } @@ -572,19 +572,19 @@ AddPreserve(XkbDescPtr xkb, if (report && (warningLevel > 0)) { char *str; - WARN2("Multiple definitions for preserve[%s] in %s\n", + WARN("Multiple definitions for preserve[%s] in %s\n", PreserveIndexTxt(type, xkb, old), TypeTxt(type)); if (clobber) str = PreserveTxt(type, xkb, new); else str = PreserveTxt(type, xkb, old); - ACTION1("Using %s, ", str); + ACTION("Using %s, ", str); if (clobber) str = PreserveTxt(type, xkb, old); else str = PreserveTxt(type, xkb, new); - INFO1("ignoring %s\n", str); + INFO("ignoring %s\n", str); } if (clobber) { @@ -596,8 +596,8 @@ AddPreserve(XkbDescPtr xkb, old = uTypedAlloc(PreserveInfo); if (!old) { - WSGO1("Couldn't allocate preserve in %s\n", TypeTxt(type)); - ACTION1("Preserve[%s] lost\n", PreserveIndexTxt(type, xkb, new)); + WSGO("Couldn't allocate preserve in %s\n", TypeTxt(type)); + ACTION("Preserve[%s] lost\n", PreserveIndexTxt(type, xkb, new)); return False; } *old = *new; @@ -638,13 +638,13 @@ AddMapEntry(XkbDescPtr xkb, use = old->level + 1; ignore = new->level + 1; } - WARN2("Multiple map entries for %s in %s\n", + WARN("Multiple map entries for %s in %s\n", MapEntryTxt(type, xkb, new), TypeTxt(type)); - ACTION2("Using %d, ignoring %d\n", use, ignore); + ACTION("Using %d, ignoring %d\n", use, ignore); } else if (warningLevel > 9) { - WARN3("Multiple occurrences of map[%s]= %d in %s\n", + WARN("Multiple occurrences of map[%s]= %d in %s\n", MapEntryTxt(type, xkb, new), new->level + 1, TypeTxt(type)); ACTION("Ignored\n"); return True; @@ -698,13 +698,13 @@ SetMapEntry(KeyTypeInfo * type, { if (warningLevel > 0) { - WARN1("Map entry for unused modifiers in %s\n", TypeTxt(type)); - ACTION1("Using %s instead of ", + WARN("Map entry for unused modifiers in %s\n", TypeTxt(type)); + ACTION("Using %s instead of ", XkbVModMaskText(type->dpy, xkb, entry.mods.real_mods & type->mask, entry.mods.vmods & type->vmask, XkbMessage)); - INFO1("%s\n", MapEntryTxt(type, xkb, &entry)); + INFO("%s\n", MapEntryTxt(type, xkb, &entry)); } entry.mods.real_mods &= type->mask; entry.mods.vmods &= type->vmask; @@ -717,9 +717,9 @@ SetMapEntry(KeyTypeInfo * type, } if ((rtrn.ival < 1) || (rtrn.ival > XkbMaxShiftLevel + 1)) { - ERROR3("Shift level %d out of range (1..%d) in key type %s\n", + ERROR("Shift level %d out of range (1..%d) in key type %s\n", XkbMaxShiftLevel + 1, rtrn.ival, TypeTxt(type)); - ACTION1("Ignoring illegal definition of map[%s]\n", + ACTION("Ignoring illegal definition of map[%s]\n", MapEntryTxt(type, xkb, &entry)); return False; } @@ -746,20 +746,20 @@ SetPreserve(KeyTypeInfo * type, { if (warningLevel > 0) { - WARN1("Preserve for modifiers not used by the %s type\n", + WARN("Preserve for modifiers not used by the %s type\n", TypeTxt(type)); - ACTION1("Index %s converted to ", + ACTION("Index %s converted to ", PreserveIndexTxt(type, xkb, &new)); } new.indexMods &= type->mask; new.indexVMods &= type->vmask; if (warningLevel > 0) - INFO1("%s\n", PreserveIndexTxt(type, xkb, &new)); + INFO("%s\n", PreserveIndexTxt(type, xkb, &new)); } if (!ExprResolveModMask(value, &rtrn, LookupVModMask, (XPointer) xkb)) { ERROR("Preserve value in a key type is not a modifier mask\n"); - ACTION2("Ignoring preserve[%s] in type %s\n", + ACTION("Ignoring preserve[%s] in type %s\n", PreserveIndexTxt(type, xkb, &new), TypeTxt(type)); return False; } @@ -770,15 +770,15 @@ SetPreserve(KeyTypeInfo * type, { if (warningLevel > 0) { - WARN2("Illegal value for preserve[%s] in type %s\n", + WARN("Illegal value for preserve[%s] in type %s\n", PreserveTxt(type, xkb, &new), TypeTxt(type)); - ACTION1("Converted %s to ", PreserveIndexTxt(type, xkb, &new)); + ACTION("Converted %s to ", PreserveIndexTxt(type, xkb, &new)); } new.preMods &= new.indexMods; new.preVMods &= new.indexVMods; if (warningLevel > 0) { - INFO1("%s\n", PreserveIndexTxt(type, xkb, &new)); + INFO("%s\n", PreserveIndexTxt(type, xkb, &new)); } } return AddPreserve(xkb, type, &new, True, True); @@ -796,7 +796,7 @@ AddLevelName(KeyTypeInfo * type, uTypedRecalloc(type->lvlNames, type->szNames, level + 1, Atom); if (type->lvlNames == NULL) { - ERROR1("Couldn't allocate level names for type %s\n", + ERROR("Couldn't allocate level names for type %s\n", TypeTxt(type)); ACTION("Level names lost\n"); type->szNames = 0; @@ -808,7 +808,7 @@ AddLevelName(KeyTypeInfo * type, { if (warningLevel > 9) { - WARN2("Duplicate names for level %d of key type %s\n", + WARN("Duplicate names for level %d of key type %s\n", level + 1, TypeTxt(type)); ACTION("Ignored\n"); } @@ -821,12 +821,12 @@ AddLevelName(KeyTypeInfo * type, char *old, *new; old = XkbAtomText(type->dpy, type->lvlNames[level], XkbMessage); new = XkbAtomText(type->dpy, name, XkbMessage); - WARN2("Multiple names for level %d of key type %s\n", + WARN("Multiple names for level %d of key type %s\n", level + 1, TypeTxt(type)); if (clobber) - ACTION2("Using %s, ignoring %s\n", new, old); + ACTION("Using %s, ignoring %s\n", new, old); else - ACTION2("Using %s, ignoring %s\n", old, new); + ACTION("Using %s, ignoring %s\n", old, new); } if (!clobber) return True; @@ -849,7 +849,7 @@ SetLevelName(KeyTypeInfo * type, ExprDef * arrayNdx, ExprDef * value) return ReportTypeBadType(type, "level name", "integer"); if ((rtrn.ival < 1) || (rtrn.ival > XkbMaxShiftLevel + 1)) { - ERROR3("Level name %d out of range (1..%d) in key type %s\n", + ERROR("Level name %d out of range (1..%d) in key type %s\n", rtrn.ival, XkbMaxShiftLevel + 1, XkbAtomText(type->dpy, type->name, XkbMessage)); @@ -859,7 +859,7 @@ SetLevelName(KeyTypeInfo * type, ExprDef * arrayNdx, ExprDef * value) level = rtrn.ival - 1; if (!ExprResolveString(value, &rtrn, NULL, NULL)) { - ERROR2("Non-string name for level %d in key type %s\n", level + 1, + ERROR("Non-string name for level %d in key type %s\n", level + 1, XkbAtomText(type->dpy, type->name, XkbMessage)); ACTION("Ignoring illegal level name definition\n"); return False; @@ -903,10 +903,10 @@ SetKeyTypeField(KeyTypeInfo * type, vmods = (tmp.uval >> 8) & 0xffff; /* xkb virtual mods */ if (type->defs.defined & _KT_Mask) { - WARN1("Multiple modifier mask definitions for key type %s\n", + WARN("Multiple modifier mask definitions for key type %s\n", XkbAtomText(type->dpy, type->name, XkbMessage)); - ACTION1("Using %s, ", TypeMaskTxt(type, xkb)); - INFO1("ignoring %s\n", XkbVModMaskText(type->dpy, xkb, mods, + ACTION("Using %s, ", TypeMaskTxt(type, xkb)); + INFO("ignoring %s\n", XkbVModMaskText(type->dpy, xkb, mods, vmods, XkbMessage)); return False; } @@ -931,7 +931,7 @@ SetKeyTypeField(KeyTypeInfo * type, type->defs.defined |= _KT_LevelNames; return SetLevelName(type, arrayNdx, value); } - ERROR2("Unknown field %s in key type %s\n", field, TypeTxt(type)); + ERROR("Unknown field %s in key type %s\n", field, TypeTxt(type)); ACTION("Definition ignored\n"); return False; } @@ -949,12 +949,12 @@ HandleKeyTypeVar(VarDef * stmt, XkbDescPtr xkb, KeyTypesInfo * info) stmt->value, info); if (elem.str != NULL) { - ERROR1("Default for unknown element %s\n", uStringText(elem.str)); - ACTION1("Value for field %s ignored\n", uStringText(field.str)); + ERROR("Default for unknown element %s\n", uStringText(elem.str)); + ACTION("Value for field %s ignored\n", uStringText(field.str)); } else if (field.str != NULL) { - ERROR1("Default defined for unknown field %s\n", + ERROR("Default defined for unknown field %s\n", uStringText(field.str)); ACTION("Ignored\n"); } @@ -1114,7 +1114,7 @@ HandleKeyTypesFile(XkbFile * file, info->errorCount++; break; default: - WSGO1("Unexpected statement type %d in HandleKeyTypesFile\n", + WSGO("Unexpected statement type %d in HandleKeyTypesFile\n", stmt->stmtType); break; } @@ -1124,7 +1124,7 @@ HandleKeyTypesFile(XkbFile * file, #ifdef NOISY ERROR("Too many errors\n"); #endif - ACTION1("Abandoning keytypes file \"%s\"\n", file->topName); + ACTION("Abandoning keytypes file \"%s\"\n", file->topName); break; } } @@ -1166,7 +1166,7 @@ CopyDefToKeyType(XkbDescPtr xkb, XkbKeyTypePtr type, KeyTypeInfo * def) if (!type->preserve) { WARN("Couldn't allocate preserve array in CopyDefToKeyType\n"); - ACTION1("Preserve setting for type %s lost\n", + ACTION("Preserve setting for type %s lost\n", XkbAtomText(def->dpy, def->name, XkbMessage)); } else @@ -1228,7 +1228,7 @@ CompileKeyTypes(XkbFile * file, XkbFileInfo * result, unsigned merge) else { WSGO("Couldn't allocate space for types name\n"); - ACTION2("Name \"%s\" (from %s) NOT assigned\n", + ACTION("Name \"%s\" (from %s) NOT assigned\n", scanFile, info.name); } } |