golang.org/x/tools/go/analysis/passes/modernize

#81775 opened 5 days ago by cszczepaniak

"invalid SuggestedFixes" bug in embedlitUnnestNeedsInvestigationToolsgoplsgopls/telemetry-wins

#81590 opened 1 week ago by adonovan

errorsastype rewrites errors.As target interface that does not implement error [1.26 backport]CherryPickApprovedTools

#81566 opened 2 weeks ago by gopherbot

errorsastype rewrites errors.As target interface that does not implement error [1.27 backport]CherryPickApprovedTools

#81389 opened 3 weeks ago by gopherbot

embedlit: comma in a comment produces invalid code [1.27 backport]BackportCherryPickApproved

#81101 opened 1 month ago by gopherbot

embedlit: comma in a comment produces invalid code [1.26 backport]

#81100 opened 1 month ago by gopherbot

embedlit: comma in a comment produces invalid codeNeedsFix

#81086 opened 1 month ago by nnnkkk7

embedlit: moving field assignments produces duplicate field name in struct literal [1.27 backport]BackportCherryPickApproved

#81059 opened 1 month ago by gopherbot

embedlit: moving field assignments produces duplicate field name in struct literal [1.26 backport]

#81058 opened 1 month ago by gopherbot

embedlit: moving field assignments produces duplicate field name in struct literalBackport

#81055 opened 1 month ago by nnnkkk7

modernizer for embedded fields in composite literalsProposal

#81017 opened 1 month ago by ProCode-Software

embedlit: drops first field incorrectlyTools

#80993 opened 1 month ago by stevenh

slicessort changes evaluation count of expressions with side effectsTools

#80844 opened 1 month ago by cuishuang

bloop deletes timer calls on unrelated testing.B valuesTools

#80841 opened 1 month ago by cuishuang

atomictypes breaks unkeyed struct literalsFixPendingTools

#80839 opened 1 month ago by cuishuang

slicesbackward suggested fix changes the value of assignment-form loop variablesTools

#80838 opened 1 month ago by cuishuang

slices.Backward fix changes semanticsTools

#80832 opened 1 month ago by anderseknert

stringsseq panics with nil pointer dereferenceTools

#80554 opened 2 months ago by blair

stringscut rewrites compound strings.Index(...) == -1 short-var-decl into non-compiling code (duplicate found variable)

#80474 opened 2 months ago by Chester13

add slices.Clip analyzer to modernize toolProposalhelp wanted

#80438 opened 2 months ago by sfllaw

slicesbackward -fix rewrites in-place s[i] mutations/method calls to a copyTools

#80410 opened 2 months ago by taran-p

errorsastype panics on package-level errors.As target varTools

#80383 opened 2 months ago by otherview

newexpr panic on zero-argument variadic callTools

#80110 opened 3 months ago by burmudar

errorsastype rewrites errors.As target interface that does not implement errorNeedsFixTools

#80031 opened 3 months ago by aethiopicuschan

errorsastype: change includes unused variable if original error was only used in errors.AsTools

#79370 opened 4 months ago by dagood

minmax: deletion of user defined functions doesn't validate that function args are actually usedBugReportTools

#79210 opened 4 months ago by aputman

improve name generation heuristic in slicesbackward modernizerFeatureRequestTools

#79150 opened 5 months ago by madelinekalil

backwardloop makes incorrect fix in some situationsBugReport

#78868 opened 5 months ago by csilvers

slicesbackward: does not handle mutating slice while iterating backwardsBugReportTools

#78829 opened 5 months ago by jakebailey

stringscut fix causes "assignment mismatch" errorBugReportNeedsInvestigation

#78643 opened 5 months ago by fengttt

slicesbackward: fix creates unreferenced local varBugReportToolshelp wanted

#78629 opened 5 months ago by adonovan

stringsbuilder: suppress diagnostics in tests, where accidentally quadratic is not a DoS riskAnalysisNeedsInvestigation

#78613 opened 5 months ago by arp242

RangeIntAnalyzer fix breaks code if type of limit expression is type parameter constrained by multiple integer typesBugReportRefactoringTools

#78571 opened 5 months ago by mwriter

add slicesbackward modernizerAnalysisLibraryProposalProposalRefactoringhelp wanted

#78484 opened 6 months ago by cszczepaniak

stringsbuilder breaks code on self-referencing codeBugReportNeedsInvestigation

#78465 opened 6 months ago by csilvers

minmax removes min and max for Go < 1.21BugReportRefactoringTools

#78250 opened 6 months ago by alexandear

slicescontains: incorrect fix deletes statements, breaks compilationAnalysisBugReport

#78149 opened 6 months ago by david-pejtersen-md

vars for atomic, embedlit, errorsastype, plusbuild, stringscut, stditeratorsProposalProposal-Acceptedrelease-blocker

#77986 opened 6 months ago by madelinekalil

stringsbuilder breaks code when GenDecl is a block declaration [1.26 backport]CherryPickApprovedTools

#77904 opened 7 months ago by gopherbot

bad rangeint rewriting [1.26 backport]CherryPickApprovedTools

#77899 opened 7 months ago by gopherbot

stringscut rewrite changes behavior [1.26 backport]CherryPickApproved

#77849 opened 7 months ago by gopherbot

fmtappendf produces uncompilable code with multi-line []byte type castBugReportTools

#77821 opened 7 months ago by JoeLancaster

stringsbuilder ignores variables if they are used multiple times [1.26 backport]CherryPickApproved

#77807 opened 7 months ago by gopherbot

waitgroup leads to a compilation error [1.26 backport]CherryPickApproved

#77805 opened 7 months ago by gopherbot

minmax breaks select statements [1.26 backport]CherryPickApproved

#77804 opened 7 months ago by gopherbot

reflect.TypeOf(nil) transformed into reflect.TypeFor[untyped nil]() [1.26 backport]CherryPickApproved

#77803 opened 7 months ago by gopherbot

stringcut "foreshadows" an existing "ok" varFixPending

#77770 opened 7 months ago by canning-duck

rangeint uses target platform's type in the range expression, breaking other platforms [1.26 backport]CherryPickApproved

#77766 opened 7 months ago by gopherbot

waitgroup leads to a compilation errorBugReportNeedsFixTools

#77754 opened 7 months ago by zhuchkovdev

stringscut rewrite changes behaviorBugReportFixPending

#77737 opened 7 months ago by stapelberg

reflecttypefor handling of TypeOf(...).Elem()FeatureRequestNeedsInvestigationTools

#77724 opened 7 months ago by dolmen

rangeint uses target platform's type in the range expression, breaking other platformsBugReportNeedsFixTools

#77691 opened 7 months ago by dagood

the `-any` flag and its documentation are misleadingBugReportDocumentationNeedsDecision

#77679 opened 7 months ago by zapateo

slicescontains produces confusing code: if slices.ContainsFunc(s, justForEffects) {}BugReportNeedsInvestigation

#77677 opened 7 months ago by fasaxc

minmax breaks select statementsBugReportNeedsFix

#77671 opened 7 months ago by canning-duck

stringsbuilder ignores variables if they are used multiple timesBugReportNeedsFix

#77659 opened 7 months ago by ccmtaylor

reflect.TypeOf(nil) transformed into reflect.TypeFor[untyped nil]()BugReportNeedsFix

#77636 opened 7 months ago by oeguchi

stringsbuilder breaks code when combining two `strings.Builder`s [1.26 backport]CherryPickApprovedTools

#77624 opened 7 months ago by gopherbot

remove the fmtappendf modernizerFeatureRequestNeedsInvestigationTools

#77581 opened 7 months ago by cespare

bloop forgets to remove b.ResetTimer() callBugReportNeedsInvestigationRefactoringTools

#77579 opened 7 months ago by adonovan

stringscut does not fully recognize variable definition reuseBugReport

#77566 opened 7 months ago by creachadair

'waitgroup' analyzer conflicts with passes/waitgroupBugReportNeedsInvestigationTools

#77560 opened 7 months ago by adonovan

waitgroup changes panic/recover semanticsNeedsInvestigationTools

#77559 opened 7 months ago by findleyr

minmax changes semantics when arguments have side effectsNeedsInvestigationTools

#77558 opened 7 months ago by findleyr

fmtappendf changes nil semantics of []byte conversionBugReportTools

#77557 opened 7 months ago by findleyr

sortslice changes NaN ordering semanticsBugReportTools

#77556 opened 7 months ago by findleyr

stringscut: OOB panic in indexArgValid analyzing "buf.Bytes()" call [1.26 backport]CherryPickApprovedTools

#77529 opened 7 months ago by gopherbot

stringscut: OOB panic in indexArgValid analyzing "buf.Bytes()" callBugReportTools

#77208 opened 8 months ago by mymmrac

bad rangeint rewritingBugReportTools

#77034 opened 9 months ago by zigo101

mapsloop breaks code with non-trivial for loopBugReportFixPendingNeedsInvestigationTools

#77008 opened 9 months ago by csilvers

stringsbuilder breaks code when combining two `strings.Builder`sBugReportNeedsFixTools

#76983 opened 9 months ago by csilvers

doesn't convert 'for i := 0; i < 10; i += 1 {}'NeedsFixTools

#76977 opened 9 months ago by strager

stringsbuilder breaks code when GenDecl is a block declarationBugReportTools

#76934 opened 9 months ago by rbrady98

stringsbuilder could suggest `fmt.Fprintf(&str, ...)`Tools

#76914 opened 9 months ago by kolyshkin

rangeint transformation is unsound when index var is implicitly used after loopBugReportTools

#76880 opened 9 months ago by adonovan

reflecttypefor: should not spell out unnamed struct typesFeatureRequestTools

#76698 opened 10 months ago by adonovan

stringscut reports false positivesBugReportToolsrelease-blocker

#76687 opened 10 months ago by ldez

omitzero: don't modify kubebuilder structsFixPendingToolsrelease-blocker

#76649 opened 10 months ago by adonovan

minmax sometimes doesn't complete the code simplification in a single passTools

#76393 opened 10 months ago by adamroyjones

mapsloop sometimes transforms valid code into invalid codeAnalysisBugReportTools

#76380 opened 10 months ago by adamroyjones

reflectypefor leads to unused local varBugReportNeedsFixRefactoringTools

#75767 opened 1 year ago by adonovan

minmax: suggest deletion of redundant min and max functionsNeedsInvestigationRefactoringToolshelp wanted

#75589 opened 1 year ago by cuiweixie

publishProposalProposal-AcceptedToolProposal

#75266 opened 1 year ago by adonovan