]> git.lizzy.rs Git - rust.git/commit
choose a less arbitrary span when parsing the empty visibility modifier
authorZack M. Davis <code@zackmdavis.net>
Sun, 1 Jul 2018 03:22:19 +0000 (20:22 -0700)
committerZack M. Davis <code@zackmdavis.net>
Sun, 1 Jul 2018 05:20:28 +0000 (22:20 -0700)
commit9df9c9df7bdcd688bcad9e91b9e2669f01d9f858
tree285a0144a736bc9b26f79873360a3ed9a2a18d6d
parent33b40f56792f0ed198b79818891634275cb8168e
choose a less arbitrary span when parsing the empty visibility modifier

Visibility spans were added to the AST in #47799 (d6bdf296) as a
`Spanned<_>`—which means that we need to choose a span even in the case
of inherited visibility (what you get when there's no `pub` &c. keyword
at all). That initial implementation's choice is pretty
counterintuitive, which could matter if we want to use it as a site to
suggest inserting a visibility modifier, &c.

(The phrase "Schelling span" in the comment is meant in analogy to the
game-theoretic concept of a "Schelling point", a value that is chosen
simply because it's what one can expect to agree upon with other agents
in the absence of explicit coördination.)
src/libsyntax/parse/parser.rs