[MPlayer-dev-eng] [PATCH] [SPELLCHECK] m_option.c
Diego Biurrun
diego at biurrun.de
Fri Aug 15 16:35:10 CEST 2003
If nobody shouts, I'll commit this tomorrow...
Diego
--- m_option.c 14 Aug 2003 12:15:14 -0000 1.19
+++ m_option.c 15 Aug 2003 14:42:16 -0000
@@ -32,7 +32,7 @@
return NULL;
}
-// Default function that just do a memcpy
+// Default function that just does a memcpy
static void copy_opt(m_option_t* opt,void* dst,void* src) {
if(dst && src)
@@ -92,7 +92,7 @@
!strcmp(param, "0")) {
if(dst) VAL(dst) = opt->min;
} else {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "invalid parameter for %s flag: %s\n",name, param);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Invalid parameter for %s flag: %s\n",name, param);
return M_OPT_INVALID;
}
return 1;
@@ -314,13 +314,13 @@
return M_OPT_MISSING_PARAM;
if ((opt->flags & M_OPT_MIN) && (strlen(param) < opt->min)) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "parameter must be >= %d chars: %s\n",
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Parameter must be >= %d chars: %s\n",
(int) opt->min, param);
return M_OPT_OUT_OF_RANGE;
}
if ((opt->flags & M_OPT_MAX) && (strlen(param) > opt->max)) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "parameter must be <= %d chars: %s\n",
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Parameter must be <= %d chars: %s\n",
(int) opt->max, param);
return M_OPT_OUT_OF_RANGE;
}
@@ -444,7 +444,7 @@
}
free(del[i]);
if(idx < 0 || idx >= ln) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Index %ld is out of range\n",idx);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Index %ld is out of range.\n",idx);
continue;
} else if(!lst[idx])
continue;
@@ -501,7 +501,7 @@
return 0;
}
- // All other op need a param
+ // All other ops need a param
if (param == NULL || strlen(param) == 0)
return M_OPT_MISSING_PARAM;
@@ -607,12 +607,12 @@
m_option_type_t m_option_type_string_list = {
"String list",
- "A list of string separated by ','\n"
- "Option with name that finish by an * allow to use the following suffix : \n"
- "\t-add : add the given parameters at the end of list\n"
- "\t-pre : add the given parameters at the begining of list\n"
- "\t-del : remove the entry at the given indexs\n"
- "\t-clr : clear the list\n"
+ "A list of strings separated by ','\n"
+ "Option with a name ending in an * permits using the following suffix: \n"
+ "\t-add: Add the given parameters at the end of the list.\n"
+ "\t-pre: Add the given parameters at the begining of the list.\n"
+ "\t-del: Remove the entry at the given indices.\n"
+ "\t-clr: Clear the list.\n"
"e.g: -vop-add flip,mirror -vop-del 2,5\n",
sizeof(char**),
M_OPT_TYPE_DYNAMIC | M_OPT_TYPE_ALLOW_WILDCARD,
@@ -955,7 +955,7 @@
return M_OPT_MISSING_PARAM;
if(!strcmp(param,"help")) {
- mp_msg(MSGT_CFGPARSER, MSGL_INFO, "Avaible formats :");
+ mp_msg(MSGT_CFGPARSER, MSGL_INFO, "Available formats:");
for(i = 0 ; mp_imgfmt_list[i].name ; i++)
mp_msg(MSGT_CFGPARSER, MSGL_INFO, " %s",mp_imgfmt_list[i].name);
mp_msg(MSGT_CFGPARSER, MSGL_INFO, "\n");
@@ -984,7 +984,7 @@
m_option_type_t m_option_type_imgfmt = {
"Image format",
- "Pls report any missing colorspace",
+ "Please report any missing colorspaces.",
sizeof(uint32_t),
0,
parse_imgfmt,
@@ -995,7 +995,7 @@
NULL
};
-//// Objects (ie filters, etc) settings
+//// Objects (i.e. filters, etc) settings
#include "m_struct.h"
@@ -1032,12 +1032,12 @@
eq[0] = '\0';
opt = m_option_list_find(desc->fields,str);
if(!opt) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: %s doesn't have a %s parameter\n",opt_name,obj_name,str);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: %s doesn't have a %s parameter.\n",opt_name,obj_name,str);
return M_OPT_UNKNOWN;
}
r = m_option_parse(opt,str,p,NULL,M_CONFIG_FILE);
if(r < 0) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: error while parsing %s parameter %s (%s)\n",opt_name,obj_name,str,p);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Error while parsing %s parameter %s (%s)\n",opt_name,obj_name,str,p);
eq[0] = '=';
return r;
}
@@ -1048,14 +1048,14 @@
eq[0] = '=';
} else {
if((*nold) >= oldmax) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: %s have only %d params, so yon can't give more than that unnamed params\n",
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: %s has only %d params, so you can't give more than %d unnamed params.\n",
opt_name,obj_name,oldmax);
return M_OPT_OUT_OF_RANGE;
}
opt = &desc->fields[(*nold)];
r = m_option_parse(opt,opt->name,str,NULL,M_CONFIG_FILE);
if(r < 0) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: error while parsing %s parameter %s (%s)\n",opt_name,obj_name,opt->name,str);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Error while parsing %s parameter %s (%s)\n",opt_name,obj_name,opt->name,str);
return r;
}
if(dst) {
@@ -1076,7 +1076,7 @@
if(!strcmp(params,"help")) { // Help
char min[50],max[50];
if(!desc->fields) {
- printf("%s doesn't have any options\n\n",name);
+ printf("%s doesn't have any options.\n\n",name);
return M_OPT_EXIT;
}
printf("\n Name Type Min Max\n\n");
@@ -1195,7 +1195,7 @@
NULL
};
-/// Some predefined types as a definition is quiet lenthy
+/// Some predefined types - a definition is quite lengthy
/// Span arguments
static m_span_t m_span_params_dflts = { -1, -1 };
@@ -1223,7 +1223,7 @@
m_obj_settings_t *ret = _ret ? *_ret : NULL;
- // Now check that the object exist
+ // Now check that the object exists
param = strchr(str,'=');
if(param) {
param[0] = '\0';
@@ -1234,7 +1234,7 @@
if(!find_obj_desc(str,list,&desc)) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: %s doesn't exist\n",opt,str);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: %s doesn't exist.\n",opt,str);
return M_OPT_INVALID;
}
@@ -1288,13 +1288,13 @@
char* endptr;
id = strtol(str_list[r],&endptr,0);
if(endptr == str_list[r]) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: invalid parameter. We a list of integer wich are the index of the elements to remove\n",opt_name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: invalid parameter. We need a list of integers wich are the indices of the elements to remove.\n",opt_name);
m_option_free(&list_opt,&str_list);
return M_OPT_INVALID;
}
if(!obj_list) continue;
if(id >= idx_max || id < -idx_max) {
- mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: index %d is out of range\n",opt_name,id);
+ mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: Index %d is out of range.\n",opt_name,id);
continue;
}
if(id < 0)
@@ -1355,7 +1355,7 @@
" Prepend the given list to the current list\n\n"
" %3$s-del x,y,...\n"
" Remove the given elements. Take the list element index (starting from 0).\n"
- " Negative index can be used (ie -1 is the last element)\n\n"
+ " Negative index can be used (i.e. -1 is the last element)\n\n"
" %3$s-clr\n"
" Clear the current list.\n",name,n,prefix);
@@ -1523,7 +1523,7 @@
char* pre_name = NULL;
if(!obj_p) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: presets need a pointer to a m_obj_presets_t in the priv field\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Presets need a pointer to a m_obj_presets_t in the priv field.\n",name);
return M_OPT_PARSER_ERR;
}
@@ -1535,7 +1535,7 @@
s = in_desc->size;
if(!strcmp(param,"help")) {
- mp_msg(MSGT_CFGPARSER, MSGL_INFO, "Avaible presets for %s->%s :",out_desc->name,name);
+ mp_msg(MSGT_CFGPARSER, MSGL_INFO, "Available presets for %s->%s:",out_desc->name,name);
for(pre = obj_p->presets;(pre_name = M_ST_MB(char*,pre,obj_p->name_off)) ;
pre += s)
mp_msg(MSGT_CFGPARSER, MSGL_ERR, " %s",pre_name);
@@ -1548,8 +1548,8 @@
if(!strcmp(pre_name,param)) break;
}
if(!pre_name) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: there no preset named %s\n"
- "Avaible presets are :",name,param);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: There is no preset named %s\n"
+ "Available presets are:",name,param);
for(pre = obj_p->presets;(pre_name = M_ST_MB(char*,pre,obj_p->name_off)) ;
pre += s)
mp_msg(MSGT_CFGPARSER, MSGL_ERR, " %s",pre_name);
@@ -1563,7 +1563,7 @@
m_option_t* out_opt = m_option_list_find(out_desc->fields,
in_desc->fields[i].name);
if(!out_opt) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: unable to find the target option for field %s.\nYou should report that to the developpers\n",name,in_desc->fields[i].name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Unable to find the target option for field %s.\nPlease report this to the developers.\n",name,in_desc->fields[i].name);
return M_OPT_PARSER_ERR;
}
m_option_copy(out_opt,M_ST_MB_P(dst,out_opt->p),M_ST_MB_P(pre,in_desc->fields[i].p));
@@ -1592,7 +1592,7 @@
m_struct_t* desc = opt->priv;
if(!desc) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: customs url need a pointer to a m_struct_t in the priv field\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Custom URL needs a pointer to a m_struct_t in the priv field.\n",name);
return M_OPT_PARSER_ERR;
}
@@ -1604,7 +1604,7 @@
m_struct_set(desc,dst,"filename",url);
return 1;
}
- mp_msg(MSGT_CFGPARSER, MSGL_ERR,"Option %s: url doesn't have a valid protocol!\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR,"Option %s: URL doesn't have a valid protocol!\n",name);
return M_OPT_INVALID;
}
pos1 = ptr1-url;
@@ -1613,7 +1613,7 @@
r = m_struct_set(desc,dst,"protocol",url);
ptr1[0] = ':';
if(r < 0) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: error while setting protocol\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Error while setting protocol.\n",name);
return r;
}
}
@@ -1634,7 +1634,7 @@
// We got something, at least a username...
int len = ptr2-ptr1;
if(!m_option_list_find(desc->fields,"username")) {
- mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: this url doesn't have a username part\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: This URL doesn't have a username part.\n",name);
// skip
} else {
ptr3 = strstr(ptr1, ":");
@@ -1642,7 +1642,7 @@
// We also have a password
int len2 = ptr2-ptr3-1;
if(!m_option_list_find(desc->fields,"password")) {
- mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: this url doesn't have a password part part\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: This URL doesn't have a password part.\n",name);
// skip
} else { // Username and password
if(dst) {
@@ -1650,14 +1650,14 @@
r = m_struct_set(desc,dst,"username",ptr1);
ptr3[0] = ':';
if(r < 0) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: error while setting username\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Error while setting username.\n",name);
return r;
}
ptr2[0] = '\0';
r = m_struct_set(desc,dst,"password",ptr3+1);
ptr2[0] = '@';
if(r < 0) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: error while setting password\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Error while setting password.\n",name);
return r;
}
}
@@ -1667,7 +1667,7 @@
r = m_struct_set(desc,dst,"username",ptr1);
ptr2[0] = '@';
if(r < 0) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: error while setting username\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Error while setting username.\n",name);
return r;
}
}
@@ -1677,7 +1677,7 @@
}
// before looking for a port number check if we have an IPv6 type numeric address
- // in IPv6 URL the numeric address should be inside square braces.
+ // in an IPv6 URL the numeric address should be inside square braces.
ptr2 = strstr(ptr1, "[");
ptr3 = strstr(ptr1, "]");
if( ptr2!=NULL && ptr3!=NULL ) {
@@ -1709,7 +1709,7 @@
// We have an URL beginning like http://www.hostname.com:1212
// Get the port number
if(!m_option_list_find(desc->fields,"port")) {
- mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: this url doesn't have a port part part\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: This URL doesn't have a port part.\n",name);
// skip
} else {
if(dst) {
@@ -1718,7 +1718,7 @@
snprintf(tmp,99,"%d",p);
r = m_struct_set(desc,dst,"port",tmp);
if(r < 0) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: error while setting port\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Error while setting port.\n",name);
return r;
}
}
@@ -1729,7 +1729,7 @@
// Get the hostname
if(pos2-pos1 > 0) {
if(!m_option_list_find(desc->fields,"hostname")) {
- mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: this url doesn't have a hostname part\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: This URL doesn't have a hostname part.\n",name);
// skip
} else {
char tmp[pos2-pos1+1];
@@ -1737,7 +1737,7 @@
tmp[pos2-pos1] = '\0';
r = m_struct_set(desc,dst,"hostname",tmp);
if(r < 0) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: error while setting hostname\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Error while setting hostname.\n",name);
return r;
}
}
@@ -1750,7 +1750,7 @@
if( strlen(ptr2)>1 ) {
// copy the path/filename in the URL container
if(!m_option_list_find(desc->fields,"filename")) {
- mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: this url doesn't have a hostname part part\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_WARN, "Option %s: This URL doesn't have a hostname part.\n",name);
// skip
} else {
if(dst) {
@@ -1764,7 +1764,7 @@
if(fname != ptr2+1)
free(fname);
if(r < 0) {
- mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: error while setting filename\n",name);
+ mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Option %s: Error while setting filename.\n",name);
return r;
}
}
More information about the MPlayer-dev-eng
mailing list