PR comment fixes

cgo/lapack.go

@@ -509,10 +509,10 @@ func (impl Implementation) Dgels(trans blas.Transpose, m, n, nrhs int, a []float
 //
 // jobU and jobVT are options for computing the singular vectors. The behavior
 // is as follows
-//  jobU == lapack.SVDAll		All M columns of U are returned in u
-//  jobU == lapack.SVDInPlace	The first min(m,n) columns are returned in u
-//  jobU == lapack.SVDOverwrite	The first min(m,n) columns of U are written into a
-//	jobU == lapack.SVDNone		The columns of U are not computed.
+//  jobU == lapack.SVDAll       All M columns of U are returned in u
+//  jobU == lapack.SVDInPlace   The first min(m,n) columns are returned in u
+//  jobU == lapack.SVDOverwrite The first min(m,n) columns of U are written into a
+//  jobU == lapack.SVDNone      The columns of U are not computed.
 // The behavior is the same for jobVT and the rows of V^T. At most one of jobU
 // and jobVT can equal lapack.SVDOverwrite.
 //

lapack.go

@@ -104,7 +104,7 @@ type SVDJob byte
 
 const (
 	SVDAll       SVDJob = 'A' // Compute all singular vectors
-	SVDInPlace          = 'S' // Compute the first singular vectors and store in provided storage.
-	SVDOverwrite        = 'O' // Compute the singular vectors and store in input matrix
+	SVDInPlace          = 'S' // Compute the first singular vectors and store them in provided storage.
+	SVDOverwrite        = 'O' // Compute the singular vectors and store them in input matrix
 	SVDNone             = 'N' // Do not compute singular vectors
 )

native/dgesvd.go

@@ -27,10 +27,10 @@ const noSVDO = "dgesvd: not coded for overwrite"
 //
 // jobU and jobVT are options for computing the singular vectors. The behavior
 // is as follows
-//  jobU == lapack.SVDAll		All M columns of U are returned in u
-//  jobU == lapack.SVDInPlace	The first min(m,n) columns are returned in u
-//  jobU == lapack.SVDOverwrite	The first min(m,n) columns of U are written into a
-//	jobU == lapack.SVDNone		The columns of U are not computed.
+//  jobU == lapack.SVDAll       All m columns of U are returned in u
+//  jobU == lapack.SVDInPlace   The first min(m,n) columns are returned in u
+//  jobU == lapack.SVDOverwrite The first min(m,n) columns of U are written into a
+//  jobU == lapack.SVDNone      The columns of U are not computed.
 // The behavior is the same for jobVT and the rows of V^T. At most one of jobU
 // and jobVT can equal lapack.SVDOverwrite.
 //
@@ -59,7 +59,6 @@ const noSVDO = "dgesvd: not coded for overwrite"
 //
 // Dgesvd returns whether the decomposition successfully completed.
 func (impl Implementation) Dgesvd(jobU, jobVT lapack.SVDJob, m, n int, a []float64, lda int, s, u []float64, ldu int, vt []float64, ldvt int, work []float64, lwork int) (ok bool) {
-
 	checkMatrix(m, n, a, lda)
 	if jobU == lapack.SVDAll {
 		checkMatrix(m, m, u, ldu)

testlapack/dgesvd.go

@@ -28,34 +28,30 @@ func DgesvdTest(t *testing.T, impl Dgesvder) {
 	for _, test := range []struct {
 		m, n, lda, ldu, ldvt int
 	}{
-		/*
-				{5, 5, 0, 0, 0},
-				{5, 6, 0, 0, 0},
-				{6, 5, 0, 0, 0},
-				{5, 9, 0, 0, 0},
-				{9, 5, 0, 0, 0},
+		{5, 5, 0, 0, 0},
+		{5, 6, 0, 0, 0},
+		{6, 5, 0, 0, 0},
+		{5, 9, 0, 0, 0},
+		{9, 5, 0, 0, 0},
 
-			{5, 5, 10, 11, 12},
-			{5, 6, 10, 11, 12},
-			{6, 5, 10, 11, 12},
-			{5, 5, 10, 11, 12},
-			{5, 9, 10, 11, 12},
-			{9, 5, 10, 11, 12},
-		*/
+		{5, 5, 10, 11, 12},
+		{5, 6, 10, 11, 12},
+		{6, 5, 10, 11, 12},
+		{5, 5, 10, 11, 12},
+		{5, 9, 10, 11, 12},
+		{9, 5, 10, 11, 12},
 
 		{300, 300, 0, 0, 0},
 		{300, 400, 0, 0, 0},
 		{400, 300, 0, 0, 0},
 		{300, 600, 0, 0, 0},
 		{600, 300, 0, 0, 0},
 
-		/*
-			{300, 300, 700, 800, 900},
-			{300, 400, 700, 800, 900},
-			{400, 300, 700, 800, 900},
-			{300, 600, 700, 800, 900},
-			{600, 300, 700, 800, 900},
-		*/
+		{300, 300, 700, 800, 900},
+		{300, 400, 700, 800, 900},
+		{400, 300, 700, 800, 900},
+		{300, 600, 700, 800, 900},
+		{600, 300, 700, 800, 900},
 	} {
 		jobU := lapack.SVDAll
 		jobVT := lapack.SVDAll