Commit 1d748921 authored by Rob Pike's avatar Rob Pike

more casifying fixups

R=rsc
DELTA=213  (0 added, 0 deleted, 213 changed)
OCL=22878
CL=22882
parent 497bb9c0
...@@ -96,125 +96,125 @@ func atob(str string) (value bool, ok bool) { ...@@ -96,125 +96,125 @@ func atob(str string) (value bool, ok bool) {
} }
type ( type (
BoolValue struct; boolValue struct;
IntValue struct; intValue struct;
Int64Value struct; int64Value struct;
UintValue struct; uintValue struct;
Uint64Value struct; uint64Value struct;
StringValue struct; stringValue struct;
) )
// -- Bool Value // -- Bool Value
type BoolValue struct { type boolValue struct {
p *bool; p *bool;
} }
func NewBoolValue(val bool, p *bool) *BoolValue { func newBoolValue(val bool, p *bool) *boolValue {
*p = val; *p = val;
return &BoolValue{p} return &boolValue{p}
} }
func (b *BoolValue) Set(val bool) { func (b *boolValue) set(val bool) {
*b.p = val; *b.p = val;
} }
func (b *BoolValue) Str() string { func (b *boolValue) str() string {
return fmt.Sprintf("%v", *b.p) return fmt.Sprintf("%v", *b.p)
} }
// -- Int Value // -- Int Value
type IntValue struct { type intValue struct {
p *int; p *int;
} }
func NewIntValue(val int, p *int) *IntValue { func newIntValue(val int, p *int) *intValue {
*p = val; *p = val;
return &IntValue{p} return &intValue{p}
} }
func (i *IntValue) Set(val int) { func (i *intValue) set(val int) {
*i.p = val; *i.p = val;
} }
func (i *IntValue) Str() string { func (i *intValue) str() string {
return fmt.Sprintf("%v", *i.p) return fmt.Sprintf("%v", *i.p)
} }
// -- Int64 Value // -- Int64 Value
type Int64Value struct { type int64Value struct {
p *int64; p *int64;
} }
func NewInt64Value(val int64, p *int64) *Int64Value { func newInt64Value(val int64, p *int64) *int64Value {
*p = val; *p = val;
return &Int64Value{p} return &int64Value{p}
} }
func (i *Int64Value) Set(val int64) { func (i *int64Value) set(val int64) {
*i.p = val; *i.p = val;
} }
func (i *Int64Value) Str() string { func (i *int64Value) str() string {
return fmt.Sprintf("%v", *i.p) return fmt.Sprintf("%v", *i.p)
} }
// -- Uint Value // -- Uint Value
type UintValue struct { type uintValue struct {
p *uint; p *uint;
} }
func NewUintValue(val uint, p *uint) *UintValue { func newUintValue(val uint, p *uint) *uintValue {
*p = val; *p = val;
return &UintValue{p} return &uintValue{p}
} }
func (i *UintValue) Set(val uint) { func (i *uintValue) set(val uint) {
*i.p = val *i.p = val
} }
func (i *UintValue) Str() string { func (i *uintValue) str() string {
return fmt.Sprintf("%v", *i.p) return fmt.Sprintf("%v", *i.p)
} }
// -- Uint64 Value // -- uint64 Value
type Uint64Value struct { type uint64Value struct {
p *uint64; p *uint64;
} }
func NewUint64Value(val uint64, p *uint64) *Uint64Value { func newUint64Value(val uint64, p *uint64) *uint64Value {
*p = val; *p = val;
return &Uint64Value{p} return &uint64Value{p}
} }
func (i *Uint64Value) Set(val uint64) { func (i *uint64Value) set(val uint64) {
*i.p = val; *i.p = val;
} }
func (i *Uint64Value) Str() string { func (i *uint64Value) str() string {
return fmt.Sprintf("%v", *i.p) return fmt.Sprintf("%v", *i.p)
} }
// -- String Value // -- string Value
type StringValue struct { type stringValue struct {
p *string; p *string;
} }
func NewStringValue(val string, p *string) *StringValue { func newStringValue(val string, p *string) *stringValue {
*p = val; *p = val;
return &StringValue{p} return &stringValue{p}
} }
func (s *StringValue) Set(val string) { func (s *stringValue) set(val string) {
*s.p = val; *s.p = val;
} }
func (s *StringValue) Str() string { func (s *stringValue) str() string {
return fmt.Sprintf("%#q", *s.p) return fmt.Sprintf("%#q", *s.p)
} }
// -- Value interface // -- Value interface
type Value interface { type Value interface {
Str() string; str() string;
} }
// -- Flag structure (internal) // -- Flag structure (internal)
...@@ -224,26 +224,26 @@ export type Flag struct { ...@@ -224,26 +224,26 @@ export type Flag struct {
value Value; value Value;
} }
type Flags struct { type allFlags struct {
actual map[string] *Flag; actual map[string] *Flag;
formal map[string] *Flag; formal map[string] *Flag;
first_arg int; first_arg int;
} }
func New() *Flags { func New() *allFlags {
f := new(Flags); f := new(allFlags);
f.first_arg = 1; // 0 is the program name, 1 is first arg f.first_arg = 1; // 0 is the program name, 1 is first arg
f.actual = make(map[string] *Flag); f.actual = make(map[string] *Flag);
f.formal = make(map[string] *Flag); f.formal = make(map[string] *Flag);
return f; return f;
} }
var flags *Flags = New(); var flags *allFlags = New();
export func PrintDefaults() { export func PrintDefaults() {
for k, f := range flags.formal { for k, f := range flags.formal {
print(" -", f.name, "=", f.value.Str(), ": ", f.usage, "\n"); print(" -", f.name, "=", f.value.str(), ": ", f.usage, "\n");
} }
} }
...@@ -273,7 +273,7 @@ export func NArg() int { ...@@ -273,7 +273,7 @@ export func NArg() int {
return sys.argc() - flags.first_arg return sys.argc() - flags.first_arg
} }
func Add(name string, value Value, usage string) { func add(name string, value Value, usage string) {
f := new(Flag); f := new(Flag);
f.name = name; f.name = name;
f.usage = usage; f.usage = usage;
...@@ -288,65 +288,65 @@ func Add(name string, value Value, usage string) { ...@@ -288,65 +288,65 @@ func Add(name string, value Value, usage string) {
export func Bool(name string, value bool, usage string) *bool { export func Bool(name string, value bool, usage string) *bool {
p := new(bool); p := new(bool);
Add(name, NewBoolValue(value, p), usage); add(name, newBoolValue(value, p), usage);
return p; return p;
} }
export func BoolVar(p *bool, name string, value bool, usage string) { export func BoolVar(p *bool, name string, value bool, usage string) {
Add(name, NewBoolValue(value, p), usage); add(name, newBoolValue(value, p), usage);
} }
export func Int(name string, value int, usage string) *int { export func Int(name string, value int, usage string) *int {
p := new(int); p := new(int);
Add(name, NewIntValue(value, p), usage); add(name, newIntValue(value, p), usage);
return p; return p;
} }
export func IntVar(p *int, name string, value int, usage string) { export func IntVar(p *int, name string, value int, usage string) {
Add(name, NewIntValue(value, p), usage); add(name, newIntValue(value, p), usage);
} }
export func Int64(name string, value int64, usage string) *int64 { export func Int64(name string, value int64, usage string) *int64 {
p := new(int64); p := new(int64);
Add(name, NewInt64Value(value, p), usage); add(name, newInt64Value(value, p), usage);
return p; return p;
} }
export func Int64Var(p *int64, name string, value int64, usage string) { export func Int64Var(p *int64, name string, value int64, usage string) {
Add(name, NewInt64Value(value, p), usage); add(name, newInt64Value(value, p), usage);
} }
export func Uint(name string, value uint, usage string) *uint { export func Uint(name string, value uint, usage string) *uint {
p := new(uint); p := new(uint);
Add(name, NewUintValue(value, p), usage); add(name, newUintValue(value, p), usage);
return p; return p;
} }
export func UintVar(p *uint, name string, value uint, usage string) { export func UintVar(p *uint, name string, value uint, usage string) {
Add(name, NewUintValue(value, p), usage); add(name, newUintValue(value, p), usage);
} }
export func Uint64(name string, value uint64, usage string) *uint64 { export func Uint64(name string, value uint64, usage string) *uint64 {
p := new(uint64); p := new(uint64);
Add(name, NewUint64Value(value, p), usage); add(name, newUint64Value(value, p), usage);
return p; return p;
} }
export func Uint64Var(p *uint64, name string, value uint64, usage string) { export func Uint64Var(p *uint64, name string, value uint64, usage string) {
Add(name, NewUint64Value(value, p), usage); add(name, newUint64Value(value, p), usage);
} }
export func String(name, value string, usage string) *string { export func String(name, value string, usage string) *string {
p := new(string); p := new(string);
Add(name, NewStringValue(value, p), usage); add(name, newStringValue(value, p), usage);
return p; return p;
} }
export func StringVar(p *string, name, value string, usage string) { export func StringVar(p *string, name, value string, usage string) {
Add(name, NewStringValue(value, p), usage); add(name, newStringValue(value, p), usage);
} }
func (f *Flags) ParseOne(index int) (ok bool, next int) func (f *allFlags) ParseOne(index int) (ok bool, next int)
{ {
s := sys.argv(index); s := sys.argv(index);
f.first_arg = index; // until proven otherwise f.first_arg = index; // until proven otherwise
...@@ -394,16 +394,16 @@ func (f *Flags) ParseOne(index int) (ok bool, next int) ...@@ -394,16 +394,16 @@ func (f *Flags) ParseOne(index int) (ok bool, next int)
print("flag provided but not defined: -", name, "\n"); print("flag provided but not defined: -", name, "\n");
Usage(); Usage();
} }
if f, ok := flag.value.(*BoolValue); ok { if f, ok := flag.value.(*boolValue); ok {
if has_value { if has_value {
k, ok := atob(value); k, ok := atob(value);
if !ok { if !ok {
print("invalid boolean value ", value, " for flag: -", name, "\n"); print("invalid boolean value ", value, " for flag: -", name, "\n");
Usage(); Usage();
} }
f.Set(k) f.set(k)
} else { } else {
f.Set(true) f.set(true)
} }
} else { } else {
// It must have a value, which might be the next argument. // It must have a value, which might be the next argument.
...@@ -417,8 +417,8 @@ func (f *Flags) ParseOne(index int) (ok bool, next int) ...@@ -417,8 +417,8 @@ func (f *Flags) ParseOne(index int) (ok bool, next int)
print("flag needs an argument: -", name, "\n"); print("flag needs an argument: -", name, "\n");
Usage(); Usage();
} }
if f, ok := flag.value.(*StringValue); ok { if f, ok := flag.value.(*stringValue); ok {
f.Set(value) f.set(value)
} else { } else {
// It's an integer flag. TODO(r): check for overflow? // It's an integer flag. TODO(r): check for overflow?
k, ok := atoi(value); k, ok := atoi(value);
...@@ -426,14 +426,14 @@ func (f *Flags) ParseOne(index int) (ok bool, next int) ...@@ -426,14 +426,14 @@ func (f *Flags) ParseOne(index int) (ok bool, next int)
print("invalid integer value ", value, " for flag: -", name, "\n"); print("invalid integer value ", value, " for flag: -", name, "\n");
Usage(); Usage();
} }
if f, ok := flag.value.(*IntValue); ok { if f, ok := flag.value.(*intValue); ok {
f.Set(int(k)); f.set(int(k));
} else if f, ok := flag.value.(*Int64Value); ok { } else if f, ok := flag.value.(*int64Value); ok {
f.Set(k); f.set(k);
} else if f, ok := flag.value.(*UintValue); ok { } else if f, ok := flag.value.(*uintValue); ok {
f.Set(uint(k)); f.set(uint(k));
} else if f, ok := flag.value.(*Uint64Value); ok { } else if f, ok := flag.value.(*uint64Value); ok {
f.Set(uint64(k)); f.set(uint64(k));
} }
} }
} }
......
This diff is collapsed.
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment