libxkbcomp: Drop unused format field for *Text functions

master
Dan Nicholson 2009-03-27 20:32:33 -07:00
parent 6a578de9cb
commit d5a9be653b
5 changed files with 20 additions and 28 deletions

View File

@ -300,8 +300,7 @@ static Bool
ReportMismatch(unsigned action, unsigned field, const char *type) ReportMismatch(unsigned action, unsigned field, const char *type)
{ {
ERROR2("Value of %s field must be of type %s\n", fieldText(field), type); ERROR2("Value of %s field must be of type %s\n", fieldText(field), type);
ACTION1("Action %s definition ignored\n", ACTION1("Action %s definition ignored\n", XkbActionTypeText(action));
XkbActionTypeText(action, XkbMessage));
return False; return False;
} }
@ -309,7 +308,7 @@ static Bool
ReportIllegal(unsigned action, unsigned field) ReportIllegal(unsigned action, unsigned field)
{ {
ERROR2("Field %s is not defined for an action of type %s\n", ERROR2("Field %s is not defined for an action of type %s\n",
fieldText(field), XkbActionTypeText(action, XkbMessage)); fieldText(field), XkbActionTypeText(action));
ACTION("Action definition ignored\n"); ACTION("Action definition ignored\n");
return False; return False;
} }
@ -318,7 +317,7 @@ static Bool
ReportActionNotArray(unsigned action, unsigned field) ReportActionNotArray(unsigned action, unsigned field)
{ {
ERROR2("The %s field in the %s action is not an array\n", ERROR2("The %s field in the %s action is not an array\n",
fieldText(field), XkbActionTypeText(action, XkbMessage)); fieldText(field), XkbActionTypeText(action));
ACTION("Action definition ignored\n"); ACTION("Action definition ignored\n");
return False; return False;
} }
@ -328,7 +327,7 @@ ReportNotFound(unsigned action, unsigned field, const char *what, char *bad)
{ {
ERROR2("%s named %s not found\n", what, bad); ERROR2("%s named %s not found\n", what, bad);
ACTION2("Ignoring the %s field of an %s action\n", fieldText(field), ACTION2("Ignoring the %s field of an %s action\n", fieldText(field),
XkbActionTypeText(action, XkbMessage)); XkbActionTypeText(action));
return False; return False;
} }
@ -500,8 +499,7 @@ CheckGroupField(unsigned action,
{ {
ERROR2("Illegal group %d (must be in the range 1..%d)\n", rtrn.ival, ERROR2("Illegal group %d (must be in the range 1..%d)\n", rtrn.ival,
XkbNumKbdGroups); XkbNumKbdGroups);
ACTION1("Action %s definition ignored\n", ACTION1("Action %s definition ignored\n", XkbActionTypeText(action));
XkbActionTypeText(action, XkbMessage));
return False; return False;
} }
if (value->op == OpNegate) if (value->op == OpNegate)

View File

@ -75,7 +75,7 @@ CompileKeymap(XkbFile *file, XkbcDescPtr xkb, unsigned merge)
break; break;
default: default:
ERROR1("Cannot compile %s alone into an XKM file\n", ERROR1("Cannot compile %s alone into an XKM file\n",
XkbConfigText(mainType, XkbMessage)); XkbConfigText(mainType));
return False; return False;
} }
have = 0; have = 0;
@ -88,16 +88,14 @@ CompileKeymap(XkbFile *file, XkbcDescPtr xkb, unsigned merge)
if ((have & (1 << file->type)) != 0) if ((have & (1 << file->type)) != 0)
{ {
ERROR2("More than one %s section in a %s file\n", ERROR2("More than one %s section in a %s file\n",
XkbConfigText(file->type, XkbMessage), XkbConfigText(file->type), XkbConfigText(mainType));
XkbConfigText(mainType, XkbMessage));
ACTION("All sections after the first ignored\n"); ACTION("All sections after the first ignored\n");
ok = False; ok = False;
} }
else if ((1 << file->type) & (~legal)) else if ((1 << file->type) & (~legal))
{ {
ERROR2("Cannot define %s in a %s file\n", ERROR2("Cannot define %s in a %s file\n",
XkbConfigText(file->type, XkbMessage), XkbConfigText(file->type), XkbConfigText(mainType));
XkbConfigText(mainType, XkbMessage));
ok = False; ok = False;
} }
else else
@ -107,8 +105,7 @@ CompileKeymap(XkbFile *file, XkbcDescPtr xkb, unsigned merge)
case XkmLayoutFile: case XkmLayoutFile:
case XkmKeymapFile: case XkmKeymapFile:
WSGO2("Illegal %s configuration in a %s file\n", WSGO2("Illegal %s configuration in a %s file\n",
XkbConfigText(file->type, XkbMessage), XkbConfigText(file->type), XkbConfigText(mainType));
XkbConfigText(mainType, XkbMessage));
ACTION("Ignored\n"); ACTION("Ignored\n");
ok = False; ok = False;
break; break;
@ -131,7 +128,7 @@ CompileKeymap(XkbFile *file, XkbcDescPtr xkb, unsigned merge)
case XkmVirtualModsIndex: case XkmVirtualModsIndex:
case XkmIndicatorsIndex: case XkmIndicatorsIndex:
WSGO1("Found an isolated %s section\n", WSGO1("Found an isolated %s section\n",
XkbConfigText(file->type, XkbMessage)); XkbConfigText(file->type));
break; break;
default: default:
WSGO1("Unknown file type %d\n", file->type); WSGO1("Unknown file type %d\n", file->type);
@ -169,13 +166,12 @@ CompileKeymap(XkbFile *file, XkbcDescPtr xkb, unsigned merge)
if (missing & bit) if (missing & bit)
{ {
ERROR2("Missing %s section in a %s file\n", ERROR2("Missing %s section in a %s file\n",
XkbConfigText(i, XkbMessage), XkbConfigText(i), XkbConfigText(mainType));
XkbConfigText(mainType, XkbMessage));
missing &= ~bit; missing &= ~bit;
} }
} }
ACTION1("Description of %s not compiled\n", ACTION1("Description of %s not compiled\n",
XkbConfigText(mainType, XkbMessage)); XkbConfigText(mainType));
ok = False; ok = False;
} }
ok = BindIndicators(xkb, True, unbound, NULL); ok = BindIndicators(xkb, True, unbound, NULL);

View File

@ -99,8 +99,7 @@ ProcessIncludeFile(IncludeStmt * stmt,
if (!mapToUse) if (!mapToUse)
{ {
ERROR3("No %s named \"%s\" in the include file \"%s\"\n", ERROR3("No %s named \"%s\" in the include file \"%s\"\n",
XkbConfigText(file_type, XkbMessage), stmt->map, XkbConfigText(file_type), stmt->map, stmt->file);
stmt->file);
ACTION("Exiting\n"); ACTION("Exiting\n");
return False; return False;
} }
@ -115,8 +114,7 @@ ProcessIncludeFile(IncludeStmt * stmt,
if (mapToUse->type != file_type) if (mapToUse->type != file_type)
{ {
ERROR2("Include file wrong type (expected %s, got %s)\n", ERROR2("Include file wrong type (expected %s, got %s)\n",
XkbConfigText(file_type, XkbMessage), XkbConfigText(file_type), XkbConfigText(mapToUse->type));
XkbConfigText(mapToUse->type, XkbMessage));
ACTION1("Include file \"%s\" ignored\n", stmt->file); ACTION1("Include file \"%s\" ignored\n", stmt->file);
return False; return False;
} }
@ -576,7 +574,7 @@ FindKeyNameForAlias(XkbcDescPtr xkb, unsigned long lname,
} }
char * char *
XkbConfigText(unsigned config, unsigned format) XkbConfigText(unsigned config)
{ {
switch (config) { switch (config) {
case XkmSemanticsFile: case XkmSemanticsFile:
@ -621,7 +619,7 @@ static char *actionTypeNames[XkbSA_NumActions]= {
}; };
char * char *
XkbActionTypeText(unsigned type, unsigned format) XkbActionTypeText(unsigned type)
{ {
if (type <= XkbSA_LastAction) if (type <= XkbSA_LastAction)

View File

@ -109,9 +109,9 @@ extern Bool FindKeyNameForAlias(XkbcDescPtr /* xkb */ ,
); );
extern char * extern char *
XkbConfigText(unsigned config, unsigned format); XkbConfigText(unsigned config);
extern char * extern char *
XkbActionTypeText(unsigned type, unsigned format); XkbActionTypeText(unsigned type);
#endif /* MISC_H */ #endif /* MISC_H */

View File

@ -449,8 +449,8 @@ MergeKeyGroups(SymbolsInfo * info,
("Multiple actions for level %d/group %d on key %s\n", ("Multiple actions for level %d/group %d on key %s\n",
i + 1, group + 1, longText(into->name, XkbMessage)); i + 1, group + 1, longText(into->name, XkbMessage));
ACTION2("Using %s, ignoring %s\n", ACTION2("Using %s, ignoring %s\n",
XkbActionTypeText(use->type, XkbMessage), XkbActionTypeText(use->type),
XkbActionTypeText(ignore->type, XkbMessage)); XkbActionTypeText(ignore->type));
} }
resultActs[i] = *use; resultActs[i] = *use;
} }