Don't try to remove leading whitespace when machine-readability isn't required

It isn't possible to determine what the whitespace convention is otherwise.

Fixes #1580.

Author: ashanbrown

pkg/golinters/nolintlint/nolintlint.go

@@ -181,21 +181,18 @@ func (l Linter) Run(fset *token.FileSet, nodes ...ast.Node) ([]Issue, error) {
 
 					// check for, report and eliminate leading spaces so we can check for other issues
 					if len(leadingSpace) > 0 {
-						machineReadableReplacement := &result.Replacement{
-							Inline: &result.InlineFix{
-								StartCol:  pos.Column - 1,
-								Length:    len(leadingSpace) + 2,
-								NewString: "//",
-							},
-						}
-
 						if (l.needs & NeedsMachineOnly) != 0 {
 							issue := NotMachine{BaseIssue: base}
-							issue.BaseIssue.replacement = machineReadableReplacement
+							issue.BaseIssue.replacement = &result.Replacement{
+								Inline: &result.InlineFix{
+									StartCol:  pos.Column - 1,
+									Length:    len(leadingSpace) + 2,
+									NewString: "//",
+								},
+							}
 							issues = append(issues, issue)
 						} else if len(leadingSpace) > 1 {
 							issue := ExtraLeadingSpace{BaseIssue: base}
-							issue.BaseIssue.replacement = machineReadableReplacement
 							issues = append(issues, issue)
 						}
 					}