Can you update the Python script in the comment right above the code and confirm that it matches your new table? It's hard for me to see what you did here based on just the patch...
On Tue, Oct 9, 2012 at 10:43 AM, Andrew Stubbs <[email protected]> wrote: > On 08/10/12 19:25, Justin Lebar wrote: >> >> I think this fails on at least one edge case: If the file contains >> only the string "__date__", then len == i == 8 and we never enter the >> loop, right? I think we in general fail to detect temporal macros at >> the very end of the file, with this patch. >> >> The solution isn't as simple as making it |i <= len|, of course, >> because the end of the loop reads str[i]. > > > Grrr, those pesky fenceposts! > > Ok, after looking at it some more, I think the correct solution is to fix > the table, not the code. > > New patch attached. > > Thanks > > Andrew > _______________________________________________ ccache mailing list [email protected] https://lists.samba.org/mailman/listinfo/ccache
