Fix exception when using tempfiles

[lhh]

import editor
ret = editor('fdsa')
Traceback (most recent call last):
File "/home/lhh/venv-38/lib/python3.8/site-packages/editor.py", line 107, in editor
filename.remove()
AttributeError: 'str' object has no attribute 'remove'

Signed-off-by: Lon Hohberger lon@metamorphism.com

[rec]

Thanks for doing this, and thanks for using this library.

And what a nice clean change that is obviously correct.

Embarrassing, too, for me. :-D I can see where I added this. And people do use this - I guess they either ignored the error, or aren't using tempfiles.

Question: why did you use os.unlink instead of os.remove? I don't care one way or the other.

Question for me: why isn't there a unit test for this code path? :-D (Well... the issue is that this library doesn't have my usual near-100% coverage because of the editor coming up. But I should have an integration test which I do by hand.)

Thanks again!

[rec]

Well, I pulled it and I'm going to make a new release immediately, no need to wait!

[rec]

Released as v1.2.1

[lhh]

Thanks! Cheers!