diff options
| author | cremno <[email protected]> | 2015-07-16 04:58:21 +0200 |
|---|---|---|
| committer | cremno <[email protected]> | 2015-07-16 04:58:21 +0200 |
| commit | 0f284091d161dbca6e07de85896cbdc9abd2de6b (patch) | |
| tree | fe737bb4403b500b2c3ccf851dfb6159533912c9 /mrbgems/mruby-bin-mruby | |
| parent | 207577f0af72874d9d643f2c46b881a9159d42d7 (diff) | |
| download | mruby-0f284091d161dbca6e07de85896cbdc9abd2de6b.tar.gz mruby-0f284091d161dbca6e07de85896cbdc9abd2de6b.zip | |
delete mrb_free()-related non-NULL checks
No need to optimize since a program only exits once and errors are rare.
Also the mruby source code doesn't have these kind of checks elsewhere.
The ones in {Time,Random}#initialize are kept because there it actually
matters
since initialization always happens and re-initialization is unlikely.
Diffstat (limited to 'mrbgems/mruby-bin-mruby')
| -rw-r--r-- | mrbgems/mruby-bin-mruby/tools/mruby/mruby.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/mrbgems/mruby-bin-mruby/tools/mruby/mruby.c b/mrbgems/mruby-bin-mruby/tools/mruby/mruby.c index 141ea151b..5ca744388 100644 --- a/mrbgems/mruby-bin-mruby/tools/mruby/mruby.c +++ b/mrbgems/mruby-bin-mruby/tools/mruby/mruby.c @@ -159,10 +159,9 @@ cleanup(mrb_state *mrb, struct _args *args) { if (args->rfp && args->rfp != stdin) fclose(args->rfp); - if (args->cmdline && !args->fname) + if (!args->fname) mrb_free(mrb, args->cmdline); - if (args->argv) - mrb_free(mrb, args->argv); + mrb_free(mrb, args->argv); mrb_close(mrb); } |