On Wed, Sep 23, 2020 at 07:10:35PM -0400, Cleber Rosa wrote: > On Wed, Sep 23, 2020 at 02:33:35PM -0400, Eduardo Habkost wrote: > > On Wed, Sep 23, 2020 at 02:29:28PM -0400, John Snow wrote: > > > On 9/23/20 11:17 AM, Eduardo Habkost wrote: > > > > This changes behavior if name=='', and I guess this is OK, but > > > > I'm not sure. I miss documentation on `visit_module()`, > > > > `visit_include()`, and `_is_user_module()`. I don't know what > > > > `name` means here, and what is a "user module". > > > > > > > > > > Good spot, I missed that. > > > > > Nice catch indeed. > > > > I can probably do: bool(name and not name.startswith('./')) > > > > > In this case I like my previous suggestion even better. :) > > - Cleber. Never mind me... I noticed that this actually gets called with None. - Cleber.