golang/go

go/ast: trailing func comments mishandled in CommentMap

Open

#22 371 ouverte le 20 oct. 2017

Voir sur GitHub
 (2 commentaires) (2 réactions) (0 assignés)Go (19 008 forks)batch import
NeedsFixhelp wanted

Métriques du dépôt

Stars
 (133 883 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Working example: https://play.golang.org/p/TlsDxVbGow

Prints: g: "trailing comment in f\n"

Seems like the correct output should either be nothing (if it is associated with _ = x), or f: "trailing comment in f\n".

What's currently happening is that the "_ = x" in func f is the most recent "important" node when we encounter the comment. Since the comment isn't on the line immediately following "_ = x", it doesn't get associated with it. We don't look further up the "important nodes" stack to notice that it's within the bounds of f. Instead, we just give up and associate it with g.

/cc @griesemer

Guide contributeur

Direction de recherche
Examinez l'implémentation de CommentMap dans go/ast ; concentrez vous sur la manière dont les commentaires de fin sont associés à la fonction englobante la plus proche. Suivez la logique qui assigne les commentaires aux nœuds et identifiez pourquoi le commentaire dans func f est incorrectement lié à g.
Stack technique
go
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Go
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30