golint fixed

Author: lunny

README.md

@@ -1,6 +1,6 @@
 # SQL builder
 
-Package builder is a simple and powerful sql builder for Go.
+Package builder is a lightweight and fast SQL builder for Go and XORM.
 
 Make sure you have installed Go 1.1+ and then:
 

builder.go

@@ -1,3 +1,7 @@
+// Copyright 2016 The Xorm Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
 package builder
 
 type optype byte
@@ -16,6 +20,7 @@ type join struct {
 	joinCond  Cond
 }
 
+// Builder describes a SQL statement
 type Builder struct {
 	optype
 	tableName string
@@ -26,41 +31,49 @@ type Builder struct {
 	updates   []Eq
 }
 
+// Select creates a select Builder
 func Select(cols ...string) *Builder {
 	builder := &Builder{cond: NewCond()}
 	return builder.Select(cols...)
 }
 
+// Insert creates an insert Builder
 func Insert(eq Eq) *Builder {
 	builder := &Builder{cond: NewCond()}
 	return builder.Insert(eq)
 }
 
+// Update creates an update Builder
 func Update(updates ...Eq) *Builder {
 	builder := &Builder{cond: NewCond()}
 	return builder.Update(updates...)
 }
 
+// Delete creates a delete Builder
 func Delete(conds ...Cond) *Builder {
 	builder := &Builder{cond: NewCond()}
 	return builder.Delete(conds...)
 }
 
+// Where sets where SQL
 func (b *Builder) Where(cond Cond) *Builder {
 	b.cond = b.cond.And(cond)
 	return b
 }
 
+// From sets the table name
 func (b *Builder) From(tableName string) *Builder {
 	b.tableName = tableName
 	return b
 }
 
+// Into sets insert table name
 func (b *Builder) Into(tableName string) *Builder {
 	b.tableName = tableName
 	return b
 }
 
+// Join sets join table and contions
 func (b *Builder) Join(joinType, joinTable string, joinCond interface{}) *Builder {
 	switch joinCond.(type) {
 	case Cond:
@@ -72,60 +85,72 @@ func (b *Builder) Join(joinType, joinTable string, joinCond interface{}) *Builde
 	return b
 }
 
+// InnerJoin sets inner join
 func (b *Builder) InnerJoin(joinTable string, joinCond interface{}) *Builder {
 	return b.Join("INNER", joinTable, joinCond)
 }
 
+// LeftJoin sets left join SQL
 func (b *Builder) LeftJoin(joinTable string, joinCond interface{}) *Builder {
 	return b.Join("LEFT", joinTable, joinCond)
 }
 
+// RightJoin sets right join SQL
 func (b *Builder) RightJoin(joinTable string, joinCond interface{}) *Builder {
 	return b.Join("RIGHT", joinTable, joinCond)
 }
 
+// CrossJoin sets cross join SQL
 func (b *Builder) CrossJoin(joinTable string, joinCond interface{}) *Builder {
 	return b.Join("CROSS", joinTable, joinCond)
 }
 
+// FullJoin sets full join SQL
 func (b *Builder) FullJoin(joinTable string, joinCond interface{}) *Builder {
 	return b.Join("FULL", joinTable, joinCond)
 }
 
+// Select sets select SQL
 func (b *Builder) Select(cols ...string) *Builder {
 	b.selects = cols
 	b.optype = selectType
 	return b
 }
 
+// And sets AND condition
 func (b *Builder) And(cond Cond) *Builder {
 	b.cond = And(b.cond, cond)
 	return b
 }
 
+// Or sets OR condition
 func (b *Builder) Or(cond Cond) *Builder {
 	b.cond = Or(b.cond, cond)
 	return b
 }
 
+// Insert sets insert SQL
 func (b *Builder) Insert(eq Eq) *Builder {
 	b.inserts = eq
 	b.optype = insertType
 	return b
 }
 
+// Update sets update SQL
 func (b *Builder) Update(updates ...Eq) *Builder {
 	b.updates = updates
 	b.optype = updateType
 	return b
 }
 
+// Delete sets delete SQL
 func (b *Builder) Delete(conds ...Cond) *Builder {
 	b.cond = b.cond.And(conds...)
 	b.optype = deleteType
 	return b
 }
 
+// WriteTo implements Writer interface
 func (b *Builder) WriteTo(w Writer) error {
 	switch b.optype {
 	case condType:

builder_delete.go

@@ -1,3 +1,7 @@
+// Copyright 2016 The Xorm Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
 package builder
 
 import (

builder_insert.go

@@ -1,3 +1,7 @@
+// Copyright 2016 The Xorm Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
 package builder
 
 import (

builder_select.go

@@ -1,3 +1,7 @@
+// Copyright 2016 The Xorm Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
 package builder
 
 import (

builder_test.go

@@ -1,3 +1,7 @@
+// Copyright 2016 The Xorm Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
 package builder
 
 import (

builder_update.go

@@ -1,3 +1,7 @@
+// Copyright 2016 The Xorm Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
 package builder
 
 import (

cond.go

@@ -1,35 +1,47 @@
+// Copyright 2016 The Xorm Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
 package builder
 
 import (
 	"bytes"
 	"io"
 )
 
+// Writer defines the interface
 type Writer interface {
 	io.Writer
 	Append(...interface{})
 }
 
-type stringWriter struct {
+var _ Writer = NewWriter()
+
+// BytesWriter implments Writer and save SQL in bytes.Buffer
+type BytesWriter struct {
 	writer *bytes.Buffer
 	buffer []byte
 	args   []interface{}
 }
 
-func NewWriter() *stringWriter {
-	w := &stringWriter{}
+// NewWriter creates a new string writer
+func NewWriter() *BytesWriter {
+	w := &BytesWriter{}
 	w.writer = bytes.NewBuffer(w.buffer)
 	return w
 }
 
-func (s *stringWriter) Write(buf []byte) (int, error) {
+// Write writes data to Writer
+func (s *BytesWriter) Write(buf []byte) (int, error) {
 	return s.writer.Write(buf)
 }
 
-func (s *stringWriter) Append(args ...interface{}) {
+// Append appends args to Writer
+func (s *BytesWriter) Append(args ...interface{}) {
 	s.args = append(s.args, args...)
 }
 
+// Cond defines an interface
 type Cond interface {
 	WriteTo(Writer) error
 	And(...Cond) Cond
@@ -41,6 +53,7 @@ type condEmpty struct{}
 
 var _ Cond = condEmpty{}
 
+// NewCond creates an empty condition
 func NewCond() Cond {
 	return condEmpty{}
 }

cond_and.go

@@ -1,3 +1,7 @@
+// Copyright 2016 The Xorm Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
 package builder
 
 import "fmt"
@@ -6,6 +10,7 @@ type condAnd []Cond
 
 var _ Cond = condAnd{}
 
+// And generates AND conditions
 func And(conds ...Cond) Cond {
 	var result = make(condAnd, 0, len(conds))
 	for _, cond := range conds {

cond_between.go

@@ -1,8 +1,12 @@
+// Copyright 2016 The Xorm Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
 package builder
 
 import "fmt"
 
-// Between
+// Between implmentes between condition
 type Between struct {
 	Col     string
 	LessVal interface{}
@@ -11,6 +15,7 @@ type Between struct {
 
 var _ Cond = Between{}
 
+// WriteTo write data to Writer
 func (between Between) WriteTo(w Writer) error {
 	if _, err := fmt.Fprintf(w, "%s BETWEEN ? AND ?", between.Col); err != nil {
 		return err
@@ -19,14 +24,17 @@ func (between Between) WriteTo(w Writer) error {
 	return nil
 }
 
+// And implments And with other conditions
 func (between Between) And(conds ...Cond) Cond {
 	return And(between, And(conds...))
 }
 
+// Or implments Or with other conditions
 func (between Between) Or(conds ...Cond) Cond {
 	return Or(between, Or(conds...))
 }
 
+// IsValid tests if the condition is valid
 func (between Between) IsValid() bool {
 	return len(between.Col) > 0
 }