darcs

Patch 1337 extract updatePatchHeader into a separat... (and 2 more)

Title extract updatePatchHeader into a separat... (and 2 more)
Superseder Nosy List ganesh
Related Issues
Status accepted Assigned To
Milestone

Created on 2015-05-29.10:45:09 by ganesh, last changed 2015-06-13.14:25:06 by bfrk.

Files
File name Status Uploaded Type Edit Remove
extract-updatepatchheader-into-a-separate-module.dpatch ganesh, 2015-05-29.10:45:09 application/x-darcs-patch
patch-preview.txt ganesh, 2015-05-29.10:45:09 text/x-darcs-patch
unnamed ganesh, 2015-05-29.10:45:09
See mailing list archives for discussion on individual patches.
Messages
msg18409 (view) Author: ganesh Date: 2015-05-29.10:45:09
Some refactorings of the patch header code.

Part of a stalled attempt to fix issue2445, but worthwhile in itself.

3 patches for repository darcs-unstable@darcs.net:screened:

Sat Mar 21 10:01:53 GMT 2015  Ganesh Sittampalam <ganesh@earth.li>
  * extract updatePatchHeader into a separate module

Sat Mar 21 10:45:57 GMT 2015  Ganesh Sittampalam <ganesh@earth.li>
  * move getLog into Darcs.UI.PatchHeader

Sat Mar 21 10:52:55 GMT 2015  Ganesh Sittampalam <ganesh@earth.li>
  * refactor updatePatchHeader a bit
  
  It only needs a repository when asking about dependencies
Attachments
msg18478 (view) Author: bfrk Date: 2015-06-13.14:25:06
All three very nice refactorings, accepted.
History
Date User Action Args
2015-05-29 10:45:09ganeshcreate
2015-06-07 19:20:11ganeshsetstatus: needs-screening -> needs-review
2015-06-13 14:25:06bfrksetstatus: needs-review -> accepted
messages: + msg18478