Remove all hardcode string to En.rc
for all file
This changes are only for ver.c
Remove hardcode dev list to cmd.rc
update mailing adress for bug report to ros-dev(a)reactos.com
sugestion by GvG
I did leave copyright year hardcode in ver.c
Modified: trunk/reactos/subsys/system/cmd/En.rc
Modified: trunk/reactos/subsys/system/cmd/cmd.rc
Modified: trunk/reactos/subsys/system/cmd/resource.h
Modified: trunk/reactos/subsys/system/cmd/type.c
Modified: trunk/reactos/subsys/system/cmd/ver.c
Modified: trunk/reactos/subsys/system/cmd/verify.c
Modified: trunk/reactos/subsys/system/cmd/vol.c
Modified: trunk/reactos/subsys/system/cmd/window.c
_____
Modified: trunk/reactos/subsys/system/cmd/En.rc
--- trunk/reactos/subsys/system/cmd/En.rc 2005-04-30 15:04:10 UTC
(rev 14876)
+++ trunk/reactos/subsys/system/cmd/En.rc 2005-04-30 15:46:59 UTC
(rev 14877)
@@ -423,8 +423,63 @@
if none of ON, OFF or /S is specified the command\n \
will toggle stopwach state\n\n"
+STRING_TYPE_HELP1, "Displays the contents of text files.\n\nTYPE
[drive:][path]filename"
+STRING_VERIFY_HELP1, "This command is just a dummy!!\n \
+Sets whether to verify that your files are written correctly to a\n \
+disk.\n\n \
+VERIFY [ON | OFF]\n\n \
+Type VERIFY without a parameter to display the current VERIFY setting."
+STRING_VERIFY_HELP2, "VERIFY is %s.\n"
+
+STRING_VERIFY_HELP3, "Must specify ON or OFF."
+
+STRING_VERSION_HELP1, "Displays shell version information\n\n \
+VER [/C][/R][/W]\n\n \
+ /C Displays credits.\n \
+ /R Displays redistribution information.\n \
+ /W Displays warranty information."
+
+STRING_VERSION_HELP2, " comes with ABSOLUTELY NO WARRANTY; for
details\n \
+type: `ver /w'. This is free software, and you are welcome to
redistribute\n \
+it under certain conditions; type `ver /r' for details. Type `ver /c'
for a\n \
+listing of credits."
+
+STRING_VERSION_HELP3, "\n This program is distributed in the hope that
it will be useful,\n \
+ but WITHOUT ANY WARRANTY; without even the implied warranty of\n \
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the\n \
+ GNU General Public License for more details."
+
+ STRING_VERSION_HELP4, "\n This program is free software; you can
redistribute it and/or modify\n \
+ it under the terms of the GNU General Public License as published by\n
\
+ the Free Software Foundation; either version 2 of the License, or\n \
+ (at your option) any later version."
+
+STRING_VERSION_HELP5, "\nSend bug reports to <ros-dev(a)reactos.com>.\n \
+Updates are available at:
http://www.reactos.com"
+
+STRING_VERSION_HELP6, "FreeDOS version written by:\n%s\nReactOS version
written by:\n%s\n"
+
+
+STRING_WINDOW_HELP1, "change console window aspect\n\n \
+WINDOW [/POS[=]left,top,width,heigth]\n \
+ [MIN|MAX|RESTORE] ['title']\n\n \
+/POS specify window placement and dimensions\n \
+MIN minimize the window\n \
+MAX maximize the window\n \
+RESTORE restore the window"
+
+STRING_WINDOW_HELP2, "change console window aspect\n\n \
+ACTIAVTE 'window' [/POS[=]left,top,width,heigth]\n \
+ [MIN|MAX|RESTORE] ['title']\n\n \
+window tile of window on wich perform actions\n \
+/POS specify window placement and dimensions\n \
+MIN minimize the window\n \
+MAX maximize the window\n \
+RESTORE restore the window\n \
+title new title"
+
STRING_CHOICE_OPTION, "YN"
STRING_COPY_OPTION, "YNA"
@@ -487,6 +542,11 @@
STRING_TIME_ERROR1, "Invalid time."
+STRING_TYPE_ERROR1, "Invalid option '/%s'\n"
+
+STRING_WINDOW_ERROR1, "window not found"
+
+
STRING_ERROR_PARAMETERF_ERROR, "Parameter format not correct - %c\n"
STRING_ERROR_INVALID_SWITCH, "Invalid switch - /%c\n"
STRING_ERROR_TOO_MANY_PARAMETERS, "Too many parameters - %s\n"
_____
Modified: trunk/reactos/subsys/system/cmd/cmd.rc
--- trunk/reactos/subsys/system/cmd/cmd.rc 2005-04-30 15:04:10 UTC
(rev 14876)
+++ trunk/reactos/subsys/system/cmd/cmd.rc 2005-04-30 15:46:59 UTC
(rev 14877)
@@ -8,9 +8,22 @@
#define REACTOS_STR_LEGAL_COPYRIGHT "Copyright (C) 1998-2001 Eric
Kohl and others\0"
#include <reactos/version.rc>
#include "En.rc"
+STRINGTABLE DISCARDABLE
+{
+ STRING_FreeDOS_DEV, " Tim Norman Matt Rains\n \
+ Evan Jeffrey Steffen Kaiser\n \
+ Svante Frey Oliver Mueller\n \
+ Aaron Kaufman Marc Desrochers\n \
+ Rob Lake John P Price\n \
+ Hans B Pufal\n"
+ STRING_REACTOS_DEV, " Eric Kohl Emanuele Aliberti\n \
+ Paolo Pantaleo Phillip Susi\n"
+
+}
+
1 ICON DISCARDABLE res/terminal.ico
_____
Modified: trunk/reactos/subsys/system/cmd/resource.h
--- trunk/reactos/subsys/system/cmd/resource.h 2005-04-30 15:04:10 UTC
(rev 14876)
+++ trunk/reactos/subsys/system/cmd/resource.h 2005-04-30 15:46:59 UTC
(rev 14877)
@@ -68,6 +68,8 @@
#define STRING_REN_ERROR1 346
#define STRING_START_ERROR1 347
#define STRING_TIME_ERROR1 348
+#define STRING_TYPE_ERROR1 349
+#define STRING_WINDOW_ERROR1 350
@@ -184,7 +186,29 @@
#define STRING_TIMER_HELP2 681
#define STRING_TIMER_HELP3 682
+#define STRING_TYPE_HELP1 683
+#define STRING_VERIFY_HELP1 684
+#define STRING_VERIFY_HELP2 685
+#define STRING_VERIFY_HELP3 686
+
+#define STRING_VERSION_HELP1 687
+#define STRING_VERSION_HELP2 688
+#define STRING_VERSION_HELP3 689
+#define STRING_VERSION_HELP4 690
+#define STRING_VERSION_HELP5 691
+#define STRING_VERSION_HELP6 692
+
+#define STRING_WINDOW_HELP1 693
+#define STRING_WINDOW_HELP2 694
+
+#define STRING_FreeDOS_DEV 800
+#define STRING_REACTOS_DEV 801
+
+
+
+
+
_____
Modified: trunk/reactos/subsys/system/cmd/type.c
--- trunk/reactos/subsys/system/cmd/type.c 2005-04-30 15:04:10 UTC
(rev 14876)
+++ trunk/reactos/subsys/system/cmd/type.c 2005-04-30 15:46:59 UTC
(rev 14877)
@@ -21,9 +21,13 @@
*
* 19-Jan-1999 (Paolo Pantaleo <paolopan(a)freemail.it>)
* Added multiple file support (copied from y.c)
+ *
+ * 30-Apr-2005 (Magnus Olsen) <magnus(a)greatlord.com>)
+ * Remove all hardcode string to En.rc
*/
#include "precomp.h"
+#include "resource.h"
#ifdef INCLUDE_CMD_TYPE
@@ -38,13 +42,14 @@
INT argc,i;
LPTSTR *argv;
LPTSTR errmsg;
+ WCHAR szMsg[RC_STRING_MAX_SIZE];
hConsoleOut=GetStdHandle (STD_OUTPUT_HANDLE);
if (!_tcsncmp (param, _T("/?"), 2))
{
- ConOutPuts (_T("Displays the contents of text
files.\n\n"
- "TYPE
[drive:][path]filename"));
+ LoadString( GetModuleHandle(NULL), STRING_TYPE_HELP1,
(LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPuts ((LPTSTR)szMsg);
return 0;
}
@@ -60,7 +65,8 @@
{
if (_T('/') == argv[i][0])
{
- ConErrPrintf(_T("Invalid option \"%S\"\n"),
argv[i] + 1);
+ LoadString( GetModuleHandle(NULL),
STRING_TYPE_ERROR1, (LPTSTR) szMsg,sizeof(szMsg));
+ ConErrPrintf ((LPTSTR)szMsg, argv[i] + 1);
continue;
}
hFile = CreateFile(argv[i],
_____
Modified: trunk/reactos/subsys/system/cmd/ver.c
--- trunk/reactos/subsys/system/cmd/ver.c 2005-04-30 15:04:10 UTC
(rev 14876)
+++ trunk/reactos/subsys/system/cmd/ver.c 2005-04-30 15:46:59 UTC
(rev 14877)
@@ -22,6 +22,7 @@
*/
#include "precomp.h"
+#include "resource.h"
VOID ShortVersion (VOID)
@@ -63,16 +64,15 @@
INT cmd_ver (LPTSTR cmd, LPTSTR param)
{
INT i;
+ WCHAR szMsg[RC_STRING_MAX_SIZE];
+ WCHAR rosdev[RC_STRING_MAX_SIZE];
+ WCHAR fredev[RC_STRING_MAX_SIZE];
if (_tcsstr (param, _T("/?")) != NULL)
{
- ConOutPuts (_T("Displays shell version information\n"
- "\n"
- "VER [/C][/R][/W]\n"
- "\n"
- " /C Displays credits.\n"
- " /R Displays redistribution
information.\n"
- " /W Displays warranty information."));
+ LoadString( GetModuleHandle(NULL), STRING_VERSION_HELP1,
(LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPuts ((LPTSTR)szMsg);
+
return 0;
}
@@ -83,11 +83,9 @@
/* Basic copyright notice */
if (param[0] == _T('\0'))
{
- ConOutPuts (_T("\n"SHELLINFO
- " comes with ABSOLUTELY NO WARRANTY; for
details\n"
- "type: `ver /w'. This is free software,
and you are welcome to redistribute\n"
- "it under certain conditions; type `ver
/r' for details. Type `ver /c' for a\n"
- "listing of credits."));
+ ConOutPuts (_T("\n"SHELLINFO));
+ LoadString( GetModuleHandle(NULL), STRING_VERSION_HELP2,
(LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPuts ((LPTSTR)szMsg);
}
else
{
@@ -111,22 +109,25 @@
if (_totupper (param[i]) == _T('W'))
{
/* Warranty notice */
- ConOutPuts (_T("\n This program is
distributed in the hope that it will be useful,\n"
- " but WITHOUT ANY
WARRANTY; without even the implied warranty of\n"
- " MERCHANTABILITY or
FITNESS FOR A PARTICULAR PURPOSE. See the\n"
- " GNU General Public
License for more details."));
+ LoadString( GetModuleHandle(NULL),
STRING_VERSION_HELP3, (LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPuts ((LPTSTR)szMsg);
}
else if (_totupper (param[i]) == _T('R'))
{
/* Redistribution notice */
- ConOutPuts (_T("\n This program is free
software; you can redistribute it and/or modify\n"
- " it under the terms of
the GNU General Public License as published by\n"
- " the Free Software
Foundation; either version 2 of the License, or\n"
- " (at your option) any
later version."));
+ LoadString( GetModuleHandle(NULL),
STRING_VERSION_HELP4, (LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPuts ((LPTSTR)szMsg);
}
else if (_totupper (param[i]) == _T('C'))
{
/* Developer listing */
+
+ LoadString( GetModuleHandle(NULL),
STRING_REACTOS_DEV, (LPTSTR) rosdev,sizeof(rosdev));
+ LoadString( GetModuleHandle(NULL), STRING_FreeDOS_DEV,
(LPTSTR) fredev,sizeof(fredev));
+ LoadString( GetModuleHandle(NULL),
STRING_VERSION_HELP6, (LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPrintf ((LPTSTR)szMsg,fredev,rosdev);
+
+ /*
ConOutPuts (_T("\n"
"FreeDOS version written
by:\n"
" Tim Norman Matt
Rains\n"
@@ -139,6 +140,7 @@
"ReactOS version written
by:\n"
" Eric Kohl
Emanuele Aliberti\n"
" Paolo Pantaleo
Phillip Susi\n"));
+ */
}
else
{
@@ -148,9 +150,8 @@
}
}
- ConOutPuts (_T("\n"
- "Send bug reports to <ekohl(a)rz-online.de>.\n"
- "Updates are available at:
http://www.reactos.com"))ot;));
+ LoadString( GetModuleHandle(NULL), STRING_VERSION_HELP5,
(LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPuts ((LPTSTR)szMsg);
return 0;
}
_____
Modified: trunk/reactos/subsys/system/cmd/verify.c
--- trunk/reactos/subsys/system/cmd/verify.c 2005-04-30 15:04:10 UTC
(rev 14876)
+++ trunk/reactos/subsys/system/cmd/verify.c 2005-04-30 15:46:59 UTC
(rev 14877)
@@ -13,9 +13,13 @@
*
* 20-Jan-1999 (Eric Kohl <ekohl(a)abo.rhein-zeitung.de>)
* Unicode and redirection ready!
+ *
+ * 30-Apr-2005 (Magnus Olsen) <magnus(a)greatlord.com>)
+ * Remove all hardcode string to En.rc
*/
#include "precomp.h"
+#include "resource.h"
#ifdef INCLUDE_CMD_VERIFY
@@ -26,24 +30,27 @@
INT cmd_verify (LPTSTR cmd, LPTSTR param)
{
+ WCHAR szMsg[RC_STRING_MAX_SIZE];
+
if (!_tcsncmp (param, _T("/?"), 2))
{
- ConOutPuts (_T("This command is just a dummy!!\n"
- "Sets whether to verify that
your files are written correctly to a\n"
- "disk.\n\n"
- "VERIFY [ON | OFF]\n\n"
- "Type VERIFY without a
parameter to display the current VERIFY setting."));
+ LoadString( GetModuleHandle(NULL), STRING_VERIFY_HELP1,
(LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPuts ((LPTSTR)szMsg);
return 0;
}
if (!*param)
- ConOutPrintf (_T("VERIFY is %s.\n"), bVerify ? D_ON :
D_OFF);
+ {
+ LoadString( GetModuleHandle(NULL), STRING_VERIFY_HELP2,
(LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPrintf ((LPTSTR)szMsg, bVerify ? D_ON : D_OFF);
+ }
else if (_tcsicmp (param, D_OFF) == 0)
bVerify = FALSE;
else if (_tcsicmp (param, D_ON) == 0)
bVerify = TRUE;
- else
- ConOutPuts (_T("Must specify ON or OFF."));
+ else
+ LoadString( GetModuleHandle(NULL), STRING_VERIFY_HELP3,
(LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPuts ((LPTSTR)szMsg);
return 0;
}
_____
Modified: trunk/reactos/subsys/system/cmd/vol.c
--- trunk/reactos/subsys/system/cmd/vol.c 2005-04-30 15:04:10 UTC
(rev 14876)
+++ trunk/reactos/subsys/system/cmd/vol.c 2005-04-30 15:46:59 UTC
(rev 14877)
@@ -21,6 +21,7 @@
*/
#include "precomp.h"
+#include "resource.h"
#ifdef INCLUDE_CMD_VOL
_____
Modified: trunk/reactos/subsys/system/cmd/window.c
--- trunk/reactos/subsys/system/cmd/window.c 2005-04-30 15:04:10 UTC
(rev 14876)
+++ trunk/reactos/subsys/system/cmd/window.c 2005-04-30 15:46:59 UTC
(rev 14877)
@@ -10,10 +10,14 @@
* 29 Sep 1999 (Paolo Pantaleo)
* activate and window in a single file using mainly the same code
* (nice size optimization :)
+ *
+ * 30-Apr-2005 (Magnus Olsen) <magnus(a)greatlord.com>)
+ * Remove all hardcode string to En.rc
*/
#include "precomp.h"
+#include "resource.h"
#if ( defined(INCLUDE_CMD_WINDOW) || defined(INCLUDE_CMD_ACTIVATE) )
@@ -173,18 +177,13 @@
INT CommandWindow (LPTSTR cmd, LPTSTR param)
{
HWND h;
+ WCHAR szMsg[RC_STRING_MAX_SIZE];
if (_tcsncmp (param, _T("/?"), 2) == 0)
{
- ConOutPuts(_T("change console window aspect\n"
- "\n"
- "WINDOW [/POS[=]left,top,width,heigth]\n"
- " [MIN|MAX|RESTORE]
[\"title\"]\n"
- "\n"
- "/POS specify window placement
and dimensions\n"
- "MIN minimize the window\n"
- "MAX maximize the window\n"
- "RESTORE restore the window"));
+ LoadString( GetModuleHandle(NULL), STRING_WINDOW_HELP1,
(LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPuts ((LPTSTR)szMsg);
+
return 0;
}
@@ -198,20 +197,12 @@
{
LPTSTR str;
HWND h;
+ WCHAR szMsg[RC_STRING_MAX_SIZE];
if (_tcsncmp (param, _T("/?"), 2) == 0)
{
- ConOutPuts(_T("change console window aspect\n"
- "\n"
- "ACTIAVTE \"window\"
[/POS[=]left,top,width,heigth]\n"
- " [MIN|MAX|RESTORE]
[\"title\"]\n"
- "\n"
- "window tile of window on wich
perform actions\n"
- "/POS specify window placement
and dimensions\n"
- "MIN minimize the window\n"
- "MAX maximize the window\n"
- "RESTORE restore the window\n"
- "title new title"));
+ LoadString( GetModuleHandle(NULL), STRING_WINDOW_HELP2,
(LPTSTR) szMsg,sizeof(szMsg));
+ ConOutPuts ((LPTSTR)szMsg);
return 0;
}
@@ -231,7 +222,8 @@
h=FindWindow(NULL, param);
if (!h)
{
- ConErrPuts("window not found");
+ LoadString( GetModuleHandle(NULL), STRING_WINDOW_ERROR1,
(LPTSTR) szMsg,sizeof(szMsg));
+ ConErrPuts ((LPTSTR)szMsg);
return 1;
}