Hi,
> > If you can provide information backing up that it works reliably,
> > patch will be accepted.
> I use the attached patch (just adding of the options to the diff
> call). After setting DPEP_DIFF_OPTIONS="-p" my C source patches have the
> function in the dpatch (see patch_c_source.diff).
> All other file types (eg. a Makefile, see patch_no_c_source.diff) have
> also a (fake) function line, but since diff ignores the stuff after
> the @@, it works.
>
> As for reliability, I think the diff tool and its options are very
> stable and tested.
> And of course it is a custom and explicitly set option. If it does
> not work then I would just not add it to DPEP_DIFF_OPTIONS.
The patch and the resulting output looks pretty neat to me.
It's a new feature, so I'd like a testsuite added to test, but
if there is no large objections from others, it looks like a
useful feature to add.
regards,
junichi
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
