Wrong usage for --hash flag is displayed after decryption #10

Closed
opened 2026-01-29 22:29:12 +00:00 by claunia · 4 comments
Owner

Originally created by @mariomadproductions on GitHub (Sep 27, 2022).

Originally assigned to: @mnadareski on GitHub.

If you use the --hash / -h flag but don't provide a path afterwards, the program only tells you of your error after the decryption/encryption is completed. And by then, the program has already read the file, so calculating the hash would take double the time. Only a minor thing, but still.

Originally created by @mariomadproductions on GitHub (Sep 27, 2022). Originally assigned to: @mnadareski on GitHub. If you use the --hash / -h flag but don't provide a path afterwards, the program only tells you of your error after the decryption/encryption is completed. And by then, the program has already read the file, so calculating the hash would take double the time. Only a minor thing, but still.
Author
Owner

@mnadareski commented on GitHub (Sep 27, 2022):

Please provide an example of your commandline. I have a feeling that you're adding the flags after the path you're decrypting, not before, as the help and readme both indicate.

@mnadareski commented on GitHub (Sep 27, 2022): Please provide an example of your commandline. I have a feeling that you're adding the flags _after_ the path you're decrypting, not _before_, as the help and readme both indicate.
Author
Owner

@mariomadproductions commented on GitHub (Sep 28, 2022):

Yeah, I think I provided it at the end.

@mariomadproductions commented on GitHub (Sep 28, 2022): Yeah, I think I provided it at the end.
Author
Owner

@mnadareski commented on GitHub (Sep 28, 2022):

Proper usage, as copied from the README: NDecrypt.exe <operation> [flags] <path> ...

@mnadareski commented on GitHub (Sep 28, 2022): Proper usage, as copied from the README: `NDecrypt.exe <operation> [flags] <path> ...`
Author
Owner

@mariomadproductions commented on GitHub (Sep 28, 2022):

Usually command line programs allow variable orders, but okay np.

@mariomadproductions commented on GitHub (Sep 28, 2022): Usually command line programs allow variable orders, but okay np.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SabreTools/NDecrypt#10