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

Reply via email to