diff options
| author | Boris Egorov <[email protected]> | 2014-09-24 13:05:49 +0700 |
|---|---|---|
| committer | Stefano Karapetsas <[email protected]> | 2014-09-30 07:56:47 +0200 |
| commit | 96fb1c780e7cebaf9fe6c4f48c8ddc5127d297d4 (patch) | |
| tree | 1d57259f425dd71d75eb66ccdfa2ffef3211d433 /libcaja-private/caja-autorun.c | |
| parent | 95eb87d34c69dc7d2596e2a4344898630e63c5bb (diff) | |
| download | caja-96fb1c780e7cebaf9fe6c4f48c8ddc5127d297d4.tar.bz2 caja-96fb1c780e7cebaf9fe6c4f48c8ddc5127d297d4.tar.xz | |
libcaja-private: fix always-false expressions
CppCheck detected a few 'always false' expressions in caja.
First two definitely appeareed due to copy-paste (mime-types
comparison). It would be nice to move x_content_type's and corresponding
greetings to some table (array of structs), wouldn't it?
The last one looks like copy-paste problem too. There were two identical
blocks of code in move_file_prepare() checking conflict type. BTW, there
is exactly the same code in the same file inside copy_move_file().
According to DRY, it would be nice to move this code to its own
function.
Diffstat (limited to 'libcaja-private/caja-autorun.c')
| -rw-r--r-- | libcaja-private/caja-autorun.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libcaja-private/caja-autorun.c b/libcaja-private/caja-autorun.c index 2896cb9d..5c7bfec8 100644 --- a/libcaja-private/caja-autorun.c +++ b/libcaja-private/caja-autorun.c @@ -1052,11 +1052,11 @@ show_dialog: { media_greeting = _("You have just inserted a blank DVD."); } - else if (strcmp (x_content_type, "x-content/blank-cd") == 0) + else if (strcmp (x_content_type, "x-content/blank-bd") == 0) { media_greeting = _("You have just inserted a blank Blu-Ray disc."); } - else if (strcmp (x_content_type, "x-content/blank-cd") == 0) + else if (strcmp (x_content_type, "x-content/blank-hddvd") == 0) { media_greeting = _("You have just inserted a blank HD DVD."); } |