|
From: | bchiara |
Subject: | Re: [Monotone-devel] binary files merging and hook |
Date: | Fri, 10 Jun 2005 07:54:05 +0200 |
User-agent: | Mozilla Thunderbird 1.0.2 (X11/20050322) |
Richard Levitte - VMS Whacker wrote:
Ok, I don't have any objections to change, but I'm curious about why you think returning true is a big mistake. That line should be called only if the file is empty or unreadable, thus it seemed safer to me assume it binary.Hi, I just looked at the recent change that has monotone automagically recognise a binary file. I pretty much like, except for one little detail: the lua hook binary_file will default to see a file as binary. I think that's a big mistake. The last line of the function should return false, not true.
Ciao, Riccardo Ps Thank you for the doc fix!
[Prev in Thread] | Current Thread | [Next in Thread] |