darcs

Patch 672 Reorganise URL fetching code. (and 3 more)

Title Reorganise URL fetching code. (and 3 more)
Superseder Nosy List kowey
Related Issues
Status accepted Assigned To
Milestone

Created on 2011-09-03.18:08:29 by kowey, last changed 2011-12-29.21:39:24 by ganesh.

Files
File name Status Uploaded Type Edit Remove
add-forgotten-url_curl-module_.dpatch kowey, 2011-09-03.19:42:00 application/x-darcs-patch
reorganise-url-fetching-code_.dpatch kowey, 2011-09-03.18:08:28 application/x-darcs-patch
unnamed kowey, 2011-09-03.18:08:28 text/x-darcs-patch
unnamed kowey, 2011-09-03.18:08:28
unnamed kowey, 2011-09-03.19:42:00 text/x-darcs-patch
unnamed kowey, 2011-09-03.19:42:00
See mailing list archives for discussion on individual patches.
Messages
msg14710 (view) Author: kowey Date: 2011-09-03.18:08:28
4 patches for repository http://darcs.net:

Just tidying up the URL stuff a bit.
Iago, the second patch is for you ;-)

Sat Sep  3 18:02:16 BST 2011  Eric Kow <kowey@darcs.net>
  * Reorganise URL fetching code.
  
  - Move everything under the URL hierarchy
  - Split curl stuff into   URL.Curl
  - Split shared types into URL.Request

Sat Sep  3 18:22:59 BST 2011  Eric Kow <kowey@darcs.net>
  * Break up urlThread into top-level functions.
  We lose the fact the helpers were internal to urlThread, but we
  probably gain in maintainability.

Sat Sep  3 18:24:20 BST 2011  Eric Kow <kowey@darcs.net>
  * Redundant liftIO.

Sat Sep  3 18:31:24 BST 2011  Eric Kow <kowey@darcs.net>
  * Slight readability tweak on URL.addReq
Attachments
msg14711 (view) Author: kowey Date: 2011-09-03.19:42:00
1 patch for repository http://darcs.net:

Sigh...

Sat Sep  3 20:30:27 BST 2011  Eric Kow <kowey@darcs.net>
  * Add forgotten URL.Curl module.
  This was moved out from URL, but I forgot to add the file
  when I created the patch.
Attachments
History
Date User Action Args
2011-09-03 18:08:29koweycreate
2011-09-03 19:42:00koweysetfiles: + unnamed, add-forgotten-url_curl-module_.dpatch, unnamed
messages: + msg14711
2011-10-12 07:03:01ganeshsetstatus: needs-screening -> needs-review
2011-12-29 21:39:24ganeshsetstatus: needs-review -> accepted