Move.c from cmd

[Mikhail Y. Zvyozdochkin]

Hi all!
Two small notes on MOVE command implementation:
1) option /Y isn't described in En.rc (and, ergo, in ALL translations);
2) NOT ALL hardcoded strings was moved from source. Yes, this ConOutPuts 
will never compiled (under #if 0), but maybe it's better to remove this 
dead code?

WBR,
   DarkHobbit

[Brandon Turner]

Hi,
These are not the only problems with move.c, there are others and it is 
documented in the subsys\system\cmd\todo.txt what else needs to be 
done(pretty much a ton of clean up).  As it is in the todo list it 
something I plan on doing, I dont have a ton of time for ROS right now 
so things in my list might not get done soon.  So if you feel up to, go 
ahead and fix move and i can answer any questions you might have.  But 
rest assured it will be fixed sometime near future.

Brandon

Mikhail Y. Zvyozdochkin wrote:
> Hi all!
> Two small notes on MOVE command implementation:
> option /Y isn't described in En.rc (and, ergo, in ALL translations);
> NOT ALL hardcoded strings was moved from source. Yes, this
>
> ConOutPuts will never compiled (under #if 0), but maybe it's better to 
> remove this dead code?
>
> WBR,
>   DarkHobbit
>
> Ros-dev mailing list
> [e-mail]
> http://www.reactos.org/mailman/listinfo/ros-dev

[Magnus Olsen]

1.
/Y yes it exists in En.rc
STRING_CHOICE_OPTION,         "YN"
STRING_COPY_OPTION,           "YNA"

2.
Hi I have remove all hardcode string from cmd, thuse string that are left is
debug string
and they will be there.

----- Original Message ----- 
From: "Mikhail Y. Zvyozdochkin" [e-mail]
To: [e-mail]
Sent: den 27 October 2005 22:01
Subject: [ros-dev] Move.c from cmd
> Hi all!
> Two small notes on MOVE command implementation:
> option /Y isn't described in En.rc (and, ergo, in ALL translations);
> NOT ALL hardcoded strings was moved from source. Yes, this ConOutPuts
>
> will never compiled (under #if 0), but maybe it's better to remove this
> dead code?
>
> WBR,
>    DarkHobbit
>
> Ros-dev mailing list
> [e-mail]
> http://www.reactos.org/mailman/listinfo/ros-dev
>
> -- 
> No virus found in this incoming message.
> Checked by AVG Free Edition.
> Version: 7.1.361 / Virus Database: 267.12.5/149 - Release Date: 2005-10-25

[Mikhail Y. Zvyozdochkin]

Hi all,
sorry for late answer, I have permanent troubles with internet...
Magnus Olsen wrote:
> /Y yes it exists in En.rc
> STRING_CHOICE_OPTION,         "YN"
> STRING_COPY_OPTION,           "YNA"

Hmmm... I didn't mean alternative lists (STRING_CHOICE_OPTION and 
STRING_COPY_OPTION). I meant HELP on MOVE command (STRING_MOVE_HELP2 
resource).

WBR,
  DarkHobbit
> <...>
> > Two small notes on MOVE command implementation:
> > option /Y isn't described in En.rc (and, ergo, in ALL translations);
> >
> > <...>
> > WBR,
> >   DarkHobbit