Sorry for the late reply. First, thanks for the comments!

> <name> section of these test definitions should be further expanded
> beyond current 'HTTP GET' description.

Claes kindly solved this comment.

> Keeping it below 80 chars adds
> gold points.

You will find a patch attached that breaks lines at 80 chars (I need
to teach my text editor to do that in the future).

> Additionally, tests/data/Makefile.am is missing references to test1109
> test1110 test1111.

As Daniel pointed out, it is really easy to forget about this update.
A warning would be really welcome so that new contributors do not fall
into the same trap.

Regards,
Julien
diff --git a/tests/data/test1110 b/tests/data/test1110
index d15d303..c7abcae 100644
--- a/tests/data/test1110
+++ b/tests/data/test1110
@@ -1,5 +1,6 @@
 <testcase>
-# Test that the fragment is not send as part of the path when the path contains a query.
+# Test that the fragment is not send as part of the path
+# when it contains a query.
 <info>
 <keywords>
 HTTP
diff --git a/tests/data/test1111 b/tests/data/test1111
index 36a2f30..dcad958 100644
--- a/tests/data/test1111
+++ b/tests/data/test1111
@@ -1,5 +1,6 @@
 <testcase>
-# Test that no fragment is not send as part of the path when the URI contains 2 '#' (does not follow RFC 2396)
+# Test that no fragment is not send as part of the path
+# when the URI contains 2 '#' (does not follow RFC 2396)
 <info>
 <keywords>
 HTTP
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html

Reply via email to