Merge pull request #138 from jfontan/fix-typos-and-warnings

Fix some typos and warnings

Author: jfontan

commits.go

@@ -28,9 +28,9 @@ func (commitsTable) Schema() sql.Schema {
 		{Name: "author_name", Type: sql.Text, Nullable: false},
 		{Name: "author_email", Type: sql.Text, Nullable: false},
 		{Name: "author_when", Type: sql.Timestamp, Nullable: false},
-		{Name: "comitter_name", Type: sql.Text, Nullable: false},
-		{Name: "comitter_email", Type: sql.Text, Nullable: false},
-		{Name: "comitter_when", Type: sql.Timestamp, Nullable: false},
+		{Name: "committer_name", Type: sql.Text, Nullable: false},
+		{Name: "committer_email", Type: sql.Text, Nullable: false},
+		{Name: "committer_when", Type: sql.Timestamp, Nullable: false},
 		{Name: "message", Type: sql.Text, Nullable: false},
 	}
 }

database.go

@@ -15,6 +15,7 @@ const (
 	remotesTableName      = "remotes"
 )
 
+// Database holds all git repository tables
 type Database struct {
 	name string
 	cr   sql.Table
@@ -26,6 +27,8 @@ type Database struct {
 	rmr  sql.Table
 }
 
+// NewDatabase creates a new Database structure and initializes its
+// tables with the given pool
 func NewDatabase(name string, pool *RepositoryPool) sql.Database {
 	return &Database{
 		name: name,
@@ -39,10 +42,12 @@ func NewDatabase(name string, pool *RepositoryPool) sql.Database {
 	}
 }
 
+// Name returns the name of the database
 func (d *Database) Name() string {
 	return d.name
 }
 
+// Tables returns a map with all initialized tables
 func (d *Database) Tables() map[string]sql.Table {
 	return map[string]sql.Table{
 		commitsTableName:      d.cr,

remotes.go

@@ -59,11 +59,11 @@ func (remotesTable) Children() []sql.Node {
 }
 
 type remotesIter struct {
-	repository_id string
-	remotes       []*git.Remote
-	conf          *gitconfig.RemoteConfig
-	remotePos     int
-	urlPos        int
+	repositoryID string
+	remotes      []*git.Remote
+	conf         *gitconfig.RemoteConfig
+	remotePos    int
+	urlPos       int
 }
 
 func (i *remotesIter) NewIterator(
@@ -75,10 +75,10 @@ func (i *remotesIter) NewIterator(
 	}
 
 	return &remotesIter{
-		repository_id: repo.ID,
-		remotes:       remotes,
-		remotePos:     0,
-		urlPos:        0}, nil
+		repositoryID: repo.ID,
+		remotes:      remotes,
+		remotePos:    0,
+		urlPos:       0}, nil
 }
 
 func (i *remotesIter) Next() (sql.Row, error) {
@@ -101,7 +101,7 @@ func (i *remotesIter) Next() (sql.Row, error) {
 	}
 
 	row := sql.NewRow(
-		i.repository_id,
+		i.repositoryID,
 		config.Name,
 		config.URLs[i.urlPos],
 		config.URLs[i.urlPos],