sebb wrote on Thu, 13 May 2021 14:29 +00:00: > On Thu, 13 May 2021 at 15:16, sebb <seb...@gmail.com> wrote: > > On Thu, 13 May 2021 at 15:11, Daniel Shahaf <d...@daniel.shahaf.name> wrote: > > > > > > sebb wrote on Wed, May 12, 2021 at 11:49:41 +0100: > > > > As the subject says > > > > > > Assuming opener.open() actually returns a URL, I don't see the problem > > > here. The variable documents the return type for anyone who may want > > > to extend the function. > > > > The problem is that it is not clear whether the variable is supposed > > to be used or not.
Then add a comment, or return the variable, etc.. > > Is it misspelt? ITYM "misspelled" ;-) > > AIUI the convention for intentionally unused variables is to prefix > > the name with an underscore, as this no longer triggers the warning in > > PyLint Last I checked, docs.python.org said (single) leading underscores designated class members that weren't part of the class's public API. > Furthermore, the method does not actually return a URL. Then the variable could be renamed. > The method behaves like the following: > https://python.readthedocs.io/en/v2.7.2/library/urllib2.html#urllib2.urlopen Thanks. Also, all the above discussion applies to revprop-change-hook.py, too. Daniel
