gnuflag

Merge lp://staging/~rogpeppe/gnuflag/001-merge-go-updates into lp://staging/gnuflag

  1. 001-merge-go-updates
  2. Merge into trunk
Proposed by Roger Peppe
Status: Merged
Merged at revision: 14
Proposed branch: lp://staging/~rogpeppe/gnuflag/001-merge-go-updates
Merge into: lp://staging/gnuflag
Diff against target: 621 lines (+192/-58)
3 files modified
export_test.go (+1/-6)
flag.go (+92/-47)
flag_test.go (+99/-5)
To merge this branch: bzr merge lp://staging/~rogpeppe/gnuflag/001-merge-go-updates
Reviewer Review Type Date Requested Status
gnuflag maintainers Pending
Review via email: mp+247632@code.staging.launchpad.net

Description of the change

merge updates from Go tree

The flag package in the Go source tree has had
a bunch of updates. This change brings gnuflag
up to date with that.

https://codereview.appspot.com/199070043/

To post a comment you must log in.
Revision history for this message
Roger Peppe (rogpeppe) wrote :

Please take a look.

Revision history for this message
Roger Peppe (rogpeppe) wrote :

Reviewers: mp+247632_code.launchpad.net,

Message:
Please take a look.

Description:
merge updates from Go tree

The flag package in the Go source tree has had
a bunch of updates. This change brings gnuflag
up to date with that.

https://code.launchpad.net/~rogpeppe/gnuflag/001-merge-go-updates/+merge/247632

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/199070043/

Affected files (+194, -58 lines):
   A [revision details]
   M export_test.go
   M flag.go
   M flag_test.go

Revision history for this message
Martin Hilton (martin-hilton) wrote :

LGTM

https://codereview.appspot.com/199070043/

Revision history for this message
Jay R. Wren (evarlast) wrote :

LGTM

https://codereview.appspot.com/199070043/

Revision history for this message
Roger Peppe (rogpeppe) wrote :

*** Submitted:

merge updates from Go tree

The flag package in the Go source tree has had
a bunch of updates. This change brings gnuflag
up to date with that.

R=martin.hilton, jay.wren
CC=
https://codereview.appspot.com/199070043

https://codereview.appspot.com/199070043/

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches