Re: [PATCH] vcs_info: Documentation update

[Daniel Shahaf <d.s@daniel.shahaf.name>]

Frank Terbeck wrote on Sun, Jan 25, 2015 at 15:20:49 +0100:
> +item(tt(pre-addon-quilt))(
> +This hook is used when tt(vcs_info)'s quilt functionality is active in "addon"
> +mode (quilt used on top of a real version control system). It is activated
> +right before any quilt specific action is taken.
> +
> +Setting the `tt(ret)' variable in this hook to a non-zero value avoids the
> +quilt specific to be run at all.

Is a word missing here?
"quilt specific" → "quilt-specific action"

> From: Daniel Shahaf <d.s@daniel.shahaf.name>
> 
> This adds documentation for more hooks and styles available in
> vcs_info:
> 
>   - patch-format
>   - nopatch-format
>   - get-unapplied
>   - pre-addon-quilt
>   - set-patch-format
> ---

Thanks for finishing what I'd started, and for applying my patches so swiftly!

Daniel

P.S. Regarding the "TODO placeholders in documentation" issue: I didn't
intend to suggest using using the manual as a todo list.  I was only saying
that if we don't document the semantics of something, then documenting its
existence is better than not documenting it at all.  Sorry for not making
myself clear.