Obviously just for discussion for now given the patch name.

This is a rather hacky attempt to fix issue1693 - unfortunately
I can't come up with anything better, at least for 2.10. I'll
add some discussion of the issues there.

1 patch for repository darcs-unstable@darcs.net:screened:

Mon Dec  8 07:15:18 GMT 2014  Ganesh Sittampalam <ganesh@earth.li>
  * WIP: sort out encoding of output

Attachments

• patch-preview.txt
• unnamed
• wip_-sort-out-encoding-of-output.dpatch

I get conflicts when I apply this to an up-to-date clone of screened.
Could you rebase it for easier review?

Here's an amended version that should apply cleanly to screened.

I'll give a few days for any final shouts of horror then will
screen it.

1 patch for repository darcs-unstable@darcs.net:screened:

Wed Dec 31 20:11:38 GMT 2014  Ganesh Sittampalam <ganesh@earth.li>
  * Make more of an effort to encode output correctly
  
  Add a new parameter to control how to encode output. This is
  really just a sticking plaster on top of an existing mess.
  
  See issue1693 for more discussion of this.

Attachments

• make-more-of-an-effort-to-encode-output-correctly.dpatch
• patch-preview.txt
• unnamed

The patch contains a few changes that were probably not meant to be in
there. I have attached a patch that rolls them back. Please review
before screening.

Attachments

• rolled-back-some-accidental-changes-to-messages-and-comments.dpatch

Oops. Thanks for the correction, I've screened it.

This patch bundle (with 1 patches) was just applied to the repository http://darcs.net/screened.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_screened.html#bundle-cd09d6d3af025c8f63652aa84bfb249ebcd678b2

I will accept the version that is in screened because

(1) it makes more transparent when we encode the data and when not,
    which is a good thing, and it forces us to think about when to
    encode and when not
(2) it allows us to quickly and more easily fix things
(3) many patches in screened already depend on it

For the long(er) run, i.e. past 2.10, see the comment I added to issue1693.

This patch bundle (with 1 patches) was just applied to the repository http://darcs.net/reviewed.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_reviewed.html#bundle-cd09d6d3af025c8f63652aa84bfb249ebcd678b2

And it goes into branch 2.10 as dependency of bundle compression optimization.