Skip to content

Conversation

@wking
Copy link
Contributor

@wking wking commented Aug 13, 2016

Found with:

$ git grep ' \*' origin/master -- specs-go/*.go | grep -v omitempty
origin/master:specs-go/config.go:       KernelTCP *uint64 `json:"kernelTCP"`
origin/master:specs-go/config.go:       ClassID *uint32 `json:"classID"`

Both settings are optional:

$ git grep '`kernelTCP`\|`classID`' origin/master
origin/master:config-linux.md:* **`kernelTCP`** *(uint64, optional)* - ...
origin/master:config-linux.md:* **`classID`** *(uint32, optional)* - ...

Found with:

  $ git grep ' \*' origin/master -- specs-go/*.go | grep -v omitempty
  origin/master:specs-go/config.go:       KernelTCP *uint64 `json:"kernelTCP"`
  origin/master:specs-go/config.go:       ClassID *uint32 `json:"classID"`

Both settings are optional:

  $ git grep '`kernelTCP`\|`classID`' origin/master
  origin/master:config-linux.md:* **`kernelTCP`** *(uint64, optional)* - ...
  origin/master:config-linux.md:* **`classID`** *(uint32, optional)* - ...

Signed-off-by: W. Trevor King <[email protected]>
@cyphar
Copy link
Member

cyphar commented Aug 16, 2016

lgtm, IANAM

@mrunalp
Copy link
Contributor

mrunalp commented Aug 17, 2016

LGTM

Approved with PullApprove

1 similar comment
@hqhq
Copy link
Contributor

hqhq commented Aug 26, 2016

LGTM

Approved with PullApprove

@hqhq hqhq merged commit 069e8e1 into opencontainers:master Aug 26, 2016
@wking wking deleted the pointer-omitempty branch August 26, 2016 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants