diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2018-12-08 08:40:03 +0100 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2018-12-08 08:40:03 +0100 |
| commit | b80cc86486c89ec2b9bcdbf013870ae1149ad9e8 (patch) | |
| tree | ca92c3798099168b4cc52b1960f37cba6c14cc3a /prog/target.go | |
| parent | 65ed24721ef645d99569b0ef4ea78932451ae5f8 (diff) | |
prog: rename ProgGen to Builder
golint suggests that "prog.Prog" is a bad naming
because everything in prog package is ProgSomething.
Rename to Builder, "prog.Builder" sounds right.
Diffstat (limited to 'prog/target.go')
| -rw-r--r-- | prog/target.go | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/prog/target.go b/prog/target.go index 56fbb860d..360e2e60f 100644 --- a/prog/target.go +++ b/prog/target.go @@ -237,14 +237,14 @@ func (g *Gen) MutateArg(arg0 Arg) (calls []*Call) { return calls } -type ProgGen struct { +type Builder struct { target *Target ma *memAlloc p *Prog } -func MakeProgGen(target *Target) *ProgGen { - return &ProgGen{ +func MakeProgGen(target *Target) *Builder { + return &Builder{ target: target, ma: newMemAlloc(target.NumPages * target.PageSize), p: &Prog{ @@ -253,24 +253,24 @@ func MakeProgGen(target *Target) *ProgGen { } } -func (pg *ProgGen) Append(c *Call) error { +func (pg *Builder) Append(c *Call) error { pg.target.assignSizesCall(c) pg.target.SanitizeCall(c) pg.p.Calls = append(pg.p.Calls, c) return nil } -func (pg *ProgGen) Allocate(size uint64) uint64 { +func (pg *Builder) Allocate(size uint64) uint64 { return pg.ma.alloc(nil, size) } -func (pg *ProgGen) AllocateVMA(npages uint64) uint64 { +func (pg *Builder) AllocateVMA(npages uint64) uint64 { psize := pg.target.PageSize addr := pg.ma.alloc(nil, (npages+1)*psize) return (addr + psize - 1) & ^(psize - 1) } -func (pg *ProgGen) Finalize() (*Prog, error) { +func (pg *Builder) Finalize() (*Prog, error) { if err := pg.p.validate(); err != nil { return nil, err } |
