inline encode err checks

This commit is contained in:
sentriz
2020-03-14 16:56:22 +00:00
parent 948327a3ac
commit acb9625f34
2 changed files with 12 additions and 14 deletions

View File

@@ -1,8 +1,6 @@
// file generated with embed tool // file generated with embed tool
// do not edit // do not edit
// //
package assets package assets
import "time" import "time"
type EmbeddedAsset struct { type EmbeddedAsset struct {

View File

@@ -52,12 +52,10 @@ func writeCmdOutput(out, cache io.Writer, pipeReader io.ReadCloser) {
break break
} }
data := buffer[0:n] data := buffer[0:n]
_, err = out.Write(data) if _, err := out.Write(data); err != nil {
if err != nil {
log.Printf("error while writing HTTP response: %s\n", err) log.Printf("error while writing HTTP response: %s\n", err)
} }
_, err = cache.Write(data) if _, err := cache.Write(data); err != nil {
if err != nil {
log.Printf("error while writing cache file: %s\n", err) log.Printf("error while writing cache file: %s\n", err)
} }
if f, ok := out.(http.Flusher); ok { if f, ok := out.(http.Flusher); ok {
@@ -72,13 +70,16 @@ func writeCmdOutput(out, cache io.Writer, pipeReader io.ReadCloser) {
// pre-format the ffmpeg command with needed options // pre-format the ffmpeg command with needed options
func ffmpegCommand(filePath string, profile *Profile, bitrate string) *exec.Cmd { func ffmpegCommand(filePath string, profile *Profile, bitrate string) *exec.Cmd {
ffmpegArgs := []string{ args := []string{
"-v", "0", "-i", filePath, "-map", "0:0", "-v", "0",
"-vn", "-b:a", bitrate, "-i", filePath,
"-map", "0:0",
"-vn",
"-b:a", bitrate,
} }
ffmpegArgs = append(ffmpegArgs, profile.ffmpegOptions...) args = append(args, profile.ffmpegOptions...)
if profile.forceRG { if profile.forceRG {
ffmpegArgs = append(ffmpegArgs, args = append(args,
// set up replaygain processing // set up replaygain processing
"-af", "volume=replaygain=track:replaygain_preamp=6dB:replaygain_noclip=0, alimiter=level=disabled", "-af", "volume=replaygain=track:replaygain_preamp=6dB:replaygain_noclip=0, alimiter=level=disabled",
// drop redundant replaygain tags // drop redundant replaygain tags
@@ -88,8 +89,8 @@ func ffmpegCommand(filePath string, profile *Profile, bitrate string) *exec.Cmd
"-metadata", "replaygain_track_peak=", "-metadata", "replaygain_track_peak=",
) )
} }
ffmpegArgs = append(ffmpegArgs, "-f", profile.Format, "-") args = append(args, "-f", profile.Format, "-")
return exec.Command("/usr/bin/ffmpeg", ffmpegArgs...) return exec.Command("/usr/bin/ffmpeg", args...)
} }
func Encode(out io.Writer, trackPath, cachePath string, profile *Profile, bitrate string) error { func Encode(out io.Writer, trackPath, cachePath string, profile *Profile, bitrate string) error {
@@ -108,7 +109,6 @@ func Encode(out io.Writer, trackPath, cachePath string, profile *Profile, bitrat
// -- @spijet // -- @spijet
// //
// start up writers for cache file and http response // start up writers for cache file and http response
// go copyCmdOutput(w, cacheFile, pipeReader)
go writeCmdOutput(out, cacheFile, pipeReader) go writeCmdOutput(out, cacheFile, pipeReader)
// run ffmpeg // run ffmpeg
if err := cmd.Run(); err != nil { if err := cmd.Run(); err != nil {