Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2024-12-28 10:04:49 +00:00
Code Issues Projects Releases Packages Wiki Activity

[CMD] todo.txt: Remove some points that have already been implemented since ages.

Browse source
This commit is contained in:
Hermès Bélusca-Maïto 2020-07-26 21:51:48 +02:00
parent 86c5e7e41f
commit ea801338f6
No known key found for this signature in database
GPG key ID: 3B2539C65E7B93D0
1 changed files with 0 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
base/shell/cmd/todo.txt
View file

@ -9,16 +9,5 @@ Not sure what is wrong with it, but probably more than just one thing blocking t
*Move.c code clean up
It works, but it needs to be cleaned up, the code is long and overly complex for what it needs to do. Also, we can remove the hack to cover for MoveFileEx bug as it isn't a bug anymore.
*If rewrite
It works decent but looks _awful_. Very hard to maintain and/or understand what the hell is going on.
*Remove Hardcoded buffers
This is mostly done thanks to Greatlord (cmd.c is the hardest spot that is left). Anytime when you are handling a string that is taken from the commandline there should be no limit to the size.
*Implement & and &&
& runs two commands no matter what.
&& runs the 2nd command only if the first was a success
Not sure where to put this code even
*Reg Testing
We need more batch files like the one Royce made for "set /a". What out for if bugs when doing this... could lead to in the wrong direction when looking for a regression.