[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
---
此致
陈子杭
武汉大学计算机学院
- [Bug-wget] Patch: Ports test for restrict-filename to python, Elita Lobo, 2015/03/14
- Re: [Bug-wget] Patch: Ports test for restrict-filename to python,
Zihang Chen <=
- Re: [Bug-wget] Patch: Ports test for restrict-filename to python, Darshit Shah, 2015/03/14
- Re: [Bug-wget] Patch: Ports test for restrict-filename to python, Elita Lobo, 2015/03/15
- Re: [Bug-wget] Patch: Ports test for restrict-filename to python, Darshit Shah, 2015/03/15
- Re: [Bug-wget] Patch: Ports test for restrict-filename to python, Elita Lobo, 2015/03/16
- Re: [Bug-wget] Patch: Ports test for restrict-filename to python, Darshit Shah, 2015/03/19
- Re: [Bug-wget] Patch: Ports test for restrict-filename to python, Elita Lobo, 2015/03/28
- Re: [Bug-wget] Patch: Ports test for restrict-filename to python, Darshit Shah, 2015/03/29