:browse seems wrong when given parameters

[yav]

The current behavior of :browse M is to see information about names defined in M from the point of view of the currently focused module. This means that you only ever get to see information about public names in M and only if you imported them.

This seems confusing. I propose that we change :browse to work as follows:

1. :browse with no parameters shows information about what's current in scope at the REPL. This includes the currently focused module, plus any dynamic bindings (e.g., let bound, or it)
2. :browse M shows all information about M, no matter what's currently in scope at the REPL.

Thoughts?

[brianhuffman]

Just to be clear: You're not proposing that :browse show information about arbitrary modules, only modules that are currently loaded; is that right? If so, then I think your proposal is totally reasonable. I didn't put much thought into the behavior of the current :browse <module> command.

[yav]

Oh yes, :browse is supposed to be read-only, so it will only work on already loaded modules.

[yav]

I've just redone how browse works in the "Nested Modules" PR. I settled on the following semantics:

1. :browse with no argument shows what's in scope at the command line
2. :browse M shows what's exported by M (no matter what's in scope at the command line)

This seems simple an consistent, and is more or less what I'd want most of the time, I think.
Also, the way the code is written, it'd be fairly easy to show what's in scope in another module, but if we deem this useful, we could use a different command

[glguy]

PR #1682 introduces :focus that can be used to change the point-of-view into a submodule. From this state you can :browse to see the internal view of a submodule.