bug-wget
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] Patch: Ports test for restrict-filename to python


From: Zihang Chen
Subject: Re: [Bug-wget] Patch: Ports test for restrict-filename to python
Date: Sun, 15 Mar 2015 00:35:29 +0800

Hi Elita,

I don't mean to be a nitpicker, but how about:
```
filename = 'SomeFile.html'
A_File = WgetFile(filename, mainpage)
B_File = WgetFile(filename.lower(), mainpage)
# also
B_File = WgetFile(filename.upper(), mainpage)
```

The rationale is to hard-code as less as possible.

I'm totally fine with the rest of the patch.

Thank you!

2015-03-14 19:25 GMT+08:00 Elita Lobo <address@hidden>:

> Hi,
>
>
> I have attached a patch below which ports the test files for
> restrict-filename to python. Kindly review it and let me know if I need to
> make any modifications.
>
>
>
>
>
>
> Thanks and Regards,
> Elita Lobo
>



-- 
Regards,
Chen Zihang,
Computer School of Wuhan University
---
此致
陈子杭
武汉大学计算机学院


reply via email to

[Prev in Thread] Current Thread [Next in Thread]