ActiveData icon indicating copy to clipboard operation
ActiveData copied to clipboard

Possible improvement for method 'html2unicode'

Open OshinoMoumou opened this issue 3 years ago • 0 comments

Greetings! I noticed that method html2unicode referred a StackOverflow post, while a recent comment to it mentioned that this raise UnicodeDecodeError with utf-8 strings. You must either decode('utf-8') it first or use xml.sax.saxutils.unescape.. I'm trying to learn the usefulness of such small updates on StackOverflow. Would this comment help improve your code? I understand that such defect might not happen in real life situation. In that case, do you think this comment can help prevent future bugs (for example, when the code were reused somewhere else)? I'll really appreciate it if you could kindly give me some feedback or suggestions. Thank you very much for your time. Have a nice day!

OshinoMoumou avatar Jun 09 '21 06:06 OshinoMoumou