News:

--

Main Menu

Miniscule trivial cleanup (typo, grammar)

Started by eumagga0x2a, July 30, 2016, 09:02:17 PM

Previous topic - Next topic

eumagga0x2a

I wonder if the capitalization of "mpeg" should be unified across the UI or at least per translation?

for i in avidemux_*.ts; do echo -e $i:  \\t Mpeg: $(grep -o Mpeg $i | wc -l)\\t MPEG: $(grep -o MPEG $i | wc -l); done
avidemux_ca.ts: Mpeg: 5 MPEG: 61
avidemux_cs.ts: Mpeg: 6 MPEG: 139
avidemux_de.ts: Mpeg: 9 MPEG: 45
avidemux_el.ts: Mpeg: 5 MPEG: 37
avidemux_en.ts: Mpeg: 1 MPEG: 12
avidemux_es.ts: Mpeg: 5 MPEG: 59
avidemux_eu.ts: Mpeg: 11 MPEG: 38
avidemux_fr.ts: Mpeg: 178 MPEG: 13
avidemux_it.ts: Mpeg: 5 MPEG: 60
avidemux_ja.ts: Mpeg: 5 MPEG: 40
avidemux_pl.ts: Mpeg: 5 MPEG: 54
avidemux_pt_BR.ts: Mpeg: 5 MPEG: 23
avidemux_ru.ts: Mpeg: 5 MPEG: 20
avidemux_sr@latin.ts: Mpeg: 5 MPEG: 42
avidemux_sr.ts: Mpeg: 5 MPEG: 42
avidemux_tr.ts: Mpeg: 5 MPEG: 36
avidemux_zh_TW.ts: Mpeg: 5 MPEG: 52


Anyway, there are minor but visible inconsistencies in the titles of available muxers in the "Output Format" dropdown list:

diff --git a/avidemux_plugins/ADM_muxers/muxerffPS/muxerffPSPlugin.cpp b/avidemux_plugins/ADM_muxers/muxerffPS/muxerffPSPlugin.cpp
index ce4d838..c2b1f13 100644
--- a/avidemux_plugins/ADM_muxers/muxerffPS/muxerffPSPlugin.cpp
+++ b/avidemux_plugins/ADM_muxers/muxerffPS/muxerffPSPlugin.cpp
@@ -25,7 +25,7 @@ ADM_MUXER_BEGIN( "mpg",muxerffPS,
                     1,0,0,
                     "ffPS",    // Internal name
                     "ffMpeg PS muxer plugin (c) Mean 2009",
-                    "Mpeg-PS Muxer( ff)", // DIsplay name
+                    "Mpeg-PS Muxer (ff)", // Display name
                     ffPSConfigure,
                     ps_muxer_param, //template
                     &psMuxerConfig, //config
diff --git a/avidemux_plugins/ADM_muxers/muxerffTS/muxerffTSPlugin.cpp b/avidemux_plugins/ADM_muxers/muxerffTS/muxerffTSPlugin.cpp
index 8b57927..3501265 100644
--- a/avidemux_plugins/ADM_muxers/muxerffTS/muxerffTSPlugin.cpp
+++ b/avidemux_plugins/ADM_muxers/muxerffTS/muxerffTSPlugin.cpp
@@ -25,7 +25,7 @@ ADM_MUXER_BEGIN( "ts",muxerffTS,
                     1,0,1,
                     "ffTS",    // Internal name
                     "ffMpeg TS muxer plugin (c) Mean 2009",
-                    "Mpeg TS Muxer (ff)", // DIsplay name
+                    "Mpeg-TS Muxer (ff)", // Display name
                     ffTSConfigure,
                     ts_muxer_param, //template
                     &tsMuxerConfig, //config


The second patch is solely at your discretion.