Axel Bonnet <axel.bonnet@ensimag.imag.fr> writes:

quoted text
> Diff_options specify whether conversion is activated or not. Blame needs
> to access these options in order to concert files with external drivers
>
> Signed-off-by: Diane Gasselin <diane.gasselin@ensimag.imag.fr>
> Signed-off-by: Clément Poulain <clement.poulain@ensimag.imag.fr>
> Signed-off-by: Axel Bonnet <axel.bonnet@ensimag.imag.fr>

The name of Clément is spelled correctly on the mail header while S-o-b
line is corrupt.  Perhaps you have recorded your commits in UTF-8 but
allowed your MUA to send in 8859-1?  This comment applies to all the
patches in the series.

quoted text
> diff --git a/builtin/blame.c b/builtin/blame.c
> index fc15863..63b497c 100644
> --- a/builtin/blame.c
> +++ b/builtin/blame.c
> @@ -89,7 +89,8 @@ struct origin {
>   * Given an origin, prepare mmfile_t structure to be used by the
>   * diff machinery
>   */
> -static void fill_origin_blob(struct origin *o, mmfile_t *file)
> +static void fill_origin_blob(struct diff_options opt,
> +			     struct origin *o, mmfile_t *file)
>  {
>  	if (!o->file.ptr) {
>  		enum object_type type;

Two points.

 * Generally we do not want to pass structures by value.  It is especially
   true when the structure is bigger than one word, and accesses to the
   variable in the callee is read-only.

 * The callee does not seem to use the new parameter yet.  You might want
   to defer this change until fill-origin-blob actually starts using it.

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html