Hi,
Thank you Joe for your suggestions.
I have incorporated the changes and attached two
files:
1)pr37145.t: This file is to be placed in
t/ssl/
2)testrev_proxy.pl: This file is to be placed
in t/htdocs/modules/cgi/
Please review.
With regards,
Zubin
--- Joe Orton <jorton@redhat.com> wrote:
> On Wed, Oct 26, 2005 at 01:30:22AM -0700, Zubin Kika
> wrote:
> > The patch is attached.
> > It contains the following files:
> > t/ssl/proxy_bug37145.t :The script file
> > t/htdocs/test.txt :Test file of size 100Kb
> > t/htdocs/test1.txt :Test file of size less than 8
> Kb
> > t/htdocs/test2.txt :Test file of size 8Kb
> > t/htdocs/modules/cgi/testrev_proxy.pl :Perl file
> that
> > will handle the posted data
> >
> > Request you to please review the same.
>
> Thanks for the contribution.
>
> - the CGI script should not rely on CGI.pm
> - call the new test simply "t/ssl/pr37145.t"
> - attach files not tarballs when posting new code
> - create a new directory for new htdocs
> (t/htdocs/modules/ssl)
> - a CGI script which outputs 100Kb of data would be
> preferable to
> adding a 100Kb file directly to the repos
> - follow the normal code style (indenting,
> whitespace etc) for the .t
> code; should be something like:
>
http://perl.apache.org/docs/2.0/devel/core/coding_style.html#Coding_Style_Guide
>
> joe
>
__________________________________
Yahoo! Mail - PC Magazine Editors' Choice 2005
http://mail.yahoo.com
|