jasder
/
runc
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Unify the errors ProcessConfigInvalid and ConfigInvalid to avoid caller confusion. 

There is no check that these are being confused since they are both in the same type.

Signed-off-by: Steve Powell <spowell@pivotal.io>
This commit is contained in:
Steve Powell 2014-09-05 10:08:11 -07:00
parent 582b25986d
commit 8453bee1ca
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
container.go
View File

@ -26,7 +26,7 @@ type Container interface {
//
// Errors:
// ContainerDestroyed - Container no longer exists,
// ProcessConfigInvalid - config is invalid,
// ConfigInvalid - config is invalid,
// ContainerPaused - Container is paused,
// SystemError - System error.
Start(config *ProcessConfig) (pid int, exitChan chan int, err Error)

3
error.go
View File

@ -8,15 +8,14 @@ const (
// Factory errors
IdInUse ErrorCode = iota
InvalidIdFormat
ConfigInvalid
// TODO: add Load errors
// Container errors
ContainerDestroyed
ProcessConfigInvalid
ContainerPaused
// Common errors
ConfigInvalid
SystemError
)