Unified Diff: abp/filters/render_script.py
Issue 29341320:
Noissue - Added flake8-abp and pep8-naming extension and fix reported warnings (Closed)
Patch Set:
Created May 12, 2016, 2:44 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« abp/filters/parser.py ('k')
|
abp/filters/renderer.py » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: abp/filters/render_script.py |
| =================================================================== |
| --- a/abp/filters/render_script.py |
| +++ b/abp/filters/render_script.py |
| @@ -14,7 +14,7 @@ |
| # along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>. |
| import argparse |
| -import codecs |
| +import io |
| import logging |
| import sys |
| @@ -55,7 +55,7 @@ |
| try: |
| lines = render_filterlist(args.infile, sources, TopSource()) |
| - with codecs.open(args.outfile, 'wb', encoding='utf-8') as out_fp: |
| + with io.open(args.outfile, 'w', encoding='utf-8') as out_fp: |
| for line in lines: |
| out_fp.write(line.to_string() + '\n') |
|
Sebastian Noack
2016/05/12 14:47:24
This will fail now if line.to_string() returns a b
Vasily Kuznetsov
2016/05/12 15:22:23
line.to_string() is supposed to always return unic
|
| except (MissingHeader, NotFound, IncludeError) as exc: |
« no previous file with comments
|
« abp/filters/parser.py ('k')
|
abp/filters/renderer.py » ('j')
|
no next file with comments »
