README: add decode CLI example #14
Conversation
There was a problem hiding this comment.
Pull request overview
This PR enhances the README documentation by adding a decode CLI example to complement the existing encode example, and improves the encode example with better shell variable quoting practices.
- Adds a commented label to the encode example for clarity
- Improves shell variable quoting by adding quotes around
$INPUT - Introduces a new decode section with complete CLI example showing how to convert silk files back to PCM/WAV format
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ffmpeg -y -i "$INPUT" -acodec pcm_s16le -f s16le -ac 1 tmp.pcm | ||
| silk_codec pts -i tmp.pcm -s $HZ -o out.silk |
There was a problem hiding this comment.
For consistency with the quoting of "$INPUT", consider also quoting "$HZ" to follow shell best practices. While numeric variables are less prone to issues, consistent quoting improves code maintainability and prevents potential issues with unexpected values.
| silk_codec stp -i "$INPUT" -o tmp.pcm -s $HZ | ||
| ffmpeg -y -ar $HZ -f s16le -i tmp.pcm -o out.wav |
There was a problem hiding this comment.
For consistency with the quoting of "$INPUT", consider also quoting "$HZ" in both commands to follow shell best practices and maintain consistency throughout the documentation.
1 participant
No description provided.