From b577e47e7c6c40e0968af4472ed9dc52e9b64f58 Mon Sep 17 00:00:00 2001 From: Joel Reicher Date: Mon, 24 Apr 2006 07:10:03 +0000 Subject: [PATCH] ssequal() does not check for a substring, but for a prefix, so added a comment to this effect. If this is sufficient, the function body should remain unchanged but the name and comments should be modified. If not, then the function needs to be corrected. --- sbr/ssequal.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/sbr/ssequal.c b/sbr/ssequal.c index 615404a..32edca9 100644 --- a/sbr/ssequal.c +++ b/sbr/ssequal.c @@ -12,6 +12,14 @@ #include /* + * THIS CODE DOES NOT WORK AS ADVERTISED. + * It is actually checking if s1 is a PREFIX of s2. + * All calls to this function need to be checked to see + * if that needs to be changed. Prefix checking is cheaper, so + * should be kept if it's sufficient. + */ + +/* * Check if s1 is a substring of s2. * If yes, then return 1, else return 0. */ -- 1.7.10.4