Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit d5ab9df

Browse files
committed
Remove unnecessary casts from size_t to int
We can use the %zu format specifier directly, no need to cast to int.
1 parent 6a1cd8b commit d5ab9df

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

src/interfaces/ecpg/preproc/ecpg.addons

+2-2
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ ECPG: stmtExecuteStmt block
3636

3737
/* It must be cut off double quotation because new_variable() double-quotes. */
3838
str[strlen(str) - 1] = '\0';
39-
sprintf(length, "%d", (int) strlen(str));
39+
sprintf(length, "%zu", strlen(str));
4040
add_variable_to_tail(&argsinsert, new_variable(str, ECPGmake_simple_type(ECPGt_const, length, 0), 0), &no_indicator);
4141
}
4242
output_statement(cat_str(3, mm_strdup("execute"), mm_strdup("$0"), $1.type), 0, ECPGst_exec_with_exprlist);
@@ -63,7 +63,7 @@ ECPG: stmtPrepareStmt block
6363

6464
/* It must be cut off double quotation because new_variable() double-quotes. */
6565
str[strlen(str) - 1] = '\0';
66-
sprintf(length, "%d", (int) strlen(str));
66+
sprintf(length, "%zu", strlen(str));
6767
add_variable_to_tail(&argsinsert, new_variable(str, ECPGmake_simple_type(ECPGt_const, length, 0), 0), &no_indicator);
6868
}
6969
output_statement(cat_str(5, mm_strdup("prepare"), mm_strdup("$0"), $1.type, mm_strdup("as"), $1.stmt), 0, ECPGst_prepare);

src/interfaces/ecpg/preproc/ecpg.trailer

+6-6
Original file line numberDiff line numberDiff line change
@@ -1099,7 +1099,7 @@ UsingValue: UsingConst
10991099
{
11001100
char *length = mm_alloc(32);
11011101

1102-
sprintf(length, "%d", (int) strlen($1));
1102+
sprintf(length, "%zu", strlen($1));
11031103
add_variable_to_head(&argsinsert, new_variable($1, ECPGmake_simple_type(ECPGt_const, length, 0), 0), &no_indicator);
11041104
}
11051105
| civar { $$ = EMPTY; }
@@ -1226,7 +1226,7 @@ IntConstVar: Iconst
12261226
{
12271227
char *length = mm_alloc(sizeof(int) * CHAR_BIT * 10 / 3);
12281228

1229-
sprintf(length, "%d", (int) strlen($1));
1229+
sprintf(length, "%zu", strlen($1));
12301230
new_variable($1, ECPGmake_simple_type(ECPGt_const, length, 0), 0);
12311231
$$ = $1;
12321232
}
@@ -1272,7 +1272,7 @@ AllConstVar: ecpg_fconst
12721272
{
12731273
char *length = mm_alloc(sizeof(int) * CHAR_BIT * 10 / 3);
12741274

1275-
sprintf(length, "%d", (int) strlen($1));
1275+
sprintf(length, "%zu", strlen($1));
12761276
new_variable($1, ECPGmake_simple_type(ECPGt_const, length, 0), 0);
12771277
$$ = $1;
12781278
}
@@ -1287,7 +1287,7 @@ AllConstVar: ecpg_fconst
12871287
char *length = mm_alloc(sizeof(int) * CHAR_BIT * 10 / 3);
12881288
char *var = cat2_str(mm_strdup("-"), $2);
12891289

1290-
sprintf(length, "%d", (int) strlen(var));
1290+
sprintf(length, "%zu", strlen(var));
12911291
new_variable(var, ECPGmake_simple_type(ECPGt_const, length, 0), 0);
12921292
$$ = var;
12931293
}
@@ -1297,7 +1297,7 @@ AllConstVar: ecpg_fconst
12971297
char *length = mm_alloc(sizeof(int) * CHAR_BIT * 10 / 3);
12981298
char *var = cat2_str(mm_strdup("-"), $2);
12991299

1300-
sprintf(length, "%d", (int) strlen(var));
1300+
sprintf(length, "%zu", strlen(var));
13011301
new_variable(var, ECPGmake_simple_type(ECPGt_const, length, 0), 0);
13021302
$$ = var;
13031303
}
@@ -1308,7 +1308,7 @@ AllConstVar: ecpg_fconst
13081308
char *var = $1 + 1;
13091309

13101310
var[strlen(var) - 1] = '\0';
1311-
sprintf(length, "%d", (int) strlen(var));
1311+
sprintf(length, "%zu", strlen(var));
13121312
new_variable(var, ECPGmake_simple_type(ECPGt_const, length, 0), 0);
13131313
$$ = var;
13141314
}

0 commit comments

Comments
 (0)