Skip to content

Commit

Permalink
fix(process): unnecessary logging option is removed
Browse files Browse the repository at this point in the history
  • Loading branch information
Roverr committed Apr 26, 2020
1 parent c9ca4cb commit cf5ff21
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
2 changes: 1 addition & 1 deletion main.go
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ func NewStream(
logrus.Error(err)
return nil, ""
}
process := NewProcess(keepFiles, audio, loggingOpts)
process := NewProcess(keepFiles, audio)
cmd := process.Spawn(path, URI)

// Create nil pointer in case logging is not enabled
Expand Down
8 changes: 3 additions & 5 deletions process.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,8 @@ type ProcessLoggingOpts struct {

// Process is the main type for creating new processes
type Process struct {
keepFiles bool
audio bool
loggingOpts ProcessLoggingOpts
keepFiles bool
audio bool
}

// Type check
Expand All @@ -35,9 +34,8 @@ var _ IProcess = (*Process)(nil)
func NewProcess(
keepFiles bool,
audio bool,
loggingOpts ProcessLoggingOpts,
) *Process {
return &Process{keepFiles, audio, loggingOpts}
return &Process{keepFiles, audio}
}

// getHLSFlags are for getting the flags based on the config context
Expand Down

0 comments on commit cf5ff21

Please sign in to comment.