On 5/14/2012 11:22 PM, Hans-Peter Nilsson wrote:
Random non-maintainer comments: I'd suggest adding a nearby comment to avoid a future edit changing it back. The attachment with the patch had the mime-type "Video/X-DV", maybe indicating an issue with your mail-client setup mismatching the .dif filename suffix.
As always, comments about what you didn't do and why you didn't do it, are often the most important (and note that code can never be self-documenting in this regard :-))
brgds, H-P