Tune a bit the extdiff toplevel comments/samples. * Capitalize the first letter of all sentences * Add an example for GNU diff(1) 'context diff' mode. * Explain the requirement for separate cmd.xxx and opts.xxx options in .hgrc, which hopefully will guard against users trying to add: [extdiff] # Add a new Mercurial command called `cdiff', which calls # GNU diff(1) in 'context diff' mode. cmd.cdiff = diff -Nprc5 which fails for recent crew builds with: $ hg cdiff . making snapshot of 1 files from rev 07dc4a569f4e making snapshot of 1 files from working dir diff -Nprc5: not found The correct way to do this is by separating the cmd.cdiff and opts.cdiff parts like this: [extdiff] # Add a new Mercurial command called `cdiff', which calls # GNU diff(1) in 'context diff' mode. cmd.cdiff = diff opts.cdiff = -Nprc5 so add it as a new example and explicitly describe it in the extdiff comments.

     1

import mercurial.util

     2

     3

def dumbdecode(s, cmd):

     4

    return s.replace('\n', '\r\n')

     5

     6

def dumbencode(s, cmd):

     7

    return s.replace('\r\n', '\n')

     8

     9

def clevertest(s, cmd):

    10

    if '\0' in s: return False

    11

    return True

    12

    13

def cleverdecode(s, cmd):

    14

    if clevertest(s, cmd):

    15

        return dumbdecode(s, cmd)

    16

    return s

    17

    18

def cleverencode(s, cmd):

    19

    if clevertest(s, cmd):

    20

        return dumbencode(s, cmd)

    21

    return s

    22

    23

mercurial.util.filtertable.update({

    24

    'dumbdecode:': dumbdecode,

    25

    'dumbencode:': dumbencode,

    26

    'cleverdecode:': cleverdecode,

    27

    'cleverencode:': cleverencode,

    28

    })