fix typo

comment should have function name as prefix

fix potential nil pointer

klog.go

@@ -284,6 +284,7 @@ func (m *moduleSpec) Get() interface{} {
 
 var errVmoduleSyntax = errors.New("syntax error: expect comma-separated list of filename=N")
 
+// Set will sets module value
 // Syntax: -vmodule=recordio=2,file=1,gfs*=3
 func (m *moduleSpec) Set(value string) error {
 	var filter []modulePat
@@ -362,6 +363,7 @@ func (t *traceLocation) Get() interface{} {
 
 var errTraceSyntax = errors.New("syntax error: expect file.go:234")
 
+// Set will sets backtrace value
 // Syntax: -log_backtrace_at=gopherflakes.go:234
 // Note that unlike vmodule the file extension is included here.
 func (t *traceLocation) Set(value string) error {

klog_test.go

@@ -647,7 +647,7 @@ func BenchmarkLogs(b *testing.B) {
 
 	testFile, err := ioutil.TempFile("", "test.log")
 	if err != nil {
-		b.Error("unable to create temporary file")
+		b.Fatal("unable to create temporary file")
 	}
 	defer os.Remove(testFile.Name())
 

klogr/klogr.go

@@ -104,7 +104,7 @@ func framesToCaller() int {
 	return 1 // something went wrong, this is safe
 }
 
-// trimDuplicates will deduplicates elements provided in multiple KV tuple
+// trimDuplicates will deduplicate elements provided in multiple KV tuple
 // slices, whilst maintaining the distinction between where the items are
 // contained.
 func trimDuplicates(kvLists ...[]interface{}) [][]interface{} {