docker/docker — pull request #8666 by blakery (2014-10-20 22:09:34+00:00)
Title: Cleanup Removed unused function GetMap in opts/opts.go

opts/opts.go:

GetMap() removed

in refactor 1ba11384bf82f824b0efbab31aaca439cfba1b4f:
* GetMap() was moved into opts.go
* A fixme tag was added questioning whether GetMap() could be removed
* All other references to GetMap() in files modifed refer to flVolumes.GetMap()

Currently:
* Only references to anything called GetMap in the entire project are to flVolumes.GetMap() in runconfig.go,
which points to opts.ValidatePath()

* Conclusion: opts.GetMap() no longer needed

GetAll "FIXME: Can we remove this?" removed:

* Fixme tag added in same commit as above.
* opts.GetAll called several times in parse.go, (196, 335, 354....)

opts.GetAll() appears to still be needed

Signed-off-by: Blake Geno