From aef295bc85bf279262e3c586eb68ec79c99b0ba6 Mon Sep 17 00:00:00 2001 From: Martin Mares Date: Mon, 30 Jun 2003 11:18:57 +0000 Subject: [PATCH] Several changes mixed to one commit (sorry, the CVS didn't work for a long time): o Changed index format ID. o MAX_COMPLEX_LEN went with the rest of complexes. o Introduced data types and handling macros for context bucket ID's. o Returned fp_hash() to its original definition -- the previous "fix" was deadly wrong: I confused indexing of bytes with indexing of words. Also, the fp_hash() has to be monotonic wrt. fpsort's order which the new one wasn't. --- lib/index.h | 37 +++++++++++++++++++++++++++++++------ 1 file changed, 31 insertions(+), 6 deletions(-) diff --git a/lib/index.h b/lib/index.h index 18066ea8..7791c889 100644 --- a/lib/index.h +++ b/lib/index.h @@ -9,10 +9,10 @@ #include SHERLOCK_CUSTOM -#define INDEX_VERSION (0x32240100+sizeof(struct card_attr)) /* Increase with each incompatible change in index format */ +#define INDEX_VERSION (0x32250100+sizeof(struct card_attr)) /* Increase with each incompatible change in index format */ /* - * Words and word complexes + * Words * * MAX_WORD_LEN is the maximum length (measured in UTF-8 characters, excluding * the terminating zero byte if there's any) of any word which may appear in the @@ -21,15 +21,39 @@ * * Caveat: If you are upcasing/downcasing the word, the UTF-8 encoding can * expand, although at most twice, so you need to reserve 2*MAX_WORD_LEN bytes. - * - * MAX_COMPLEX_LEN is the upper bound on number of words in any word complex. */ #define MAX_WORD_LEN 64 /* a multiple of 4 */ -#define MAX_COMPLEX_LEN 10 /* Word and string types are defined in lib/custom.h */ +/* Types used for storing contexts */ + +#ifdef CONFIG_CONTEXTS +#if CONFIG_MAX_CONTEXTS == 32768 +typedef u16 context_t; +#define bget_context bgetw +#define bput_context bputw +#define GET_CONTEXT GET_U16 +#define PUT_CONTEXT PUT_U16 +#elif CONFIG_MAX_CONTEXTS == 256 +typedef byte context_t; +#define bget_context bgetc +#define bput_context bputc +#define GET_CONTEXT GET_U8 +#define PUT_CONTEXT PUT_U8 +#else +#error CONFIG_MAX_CONTEXTS set to an invalid value. +#endif +#else +struct fastbuf; +typedef struct { } context_t; +static inline uns bget_context(struct fastbuf *b UNUSED) { return 0; } +static inline void bput_context(struct fastbuf *b UNUSED, uns context UNUSED) { } +#define GET_CONTEXT(p) 0 +#define PUT_CONTEXT(p,x) do {} while(0) +#endif + /* Index card attributes */ struct card_attr { @@ -103,7 +127,8 @@ void fingerprint(byte *string, struct fingerprint *fp); static inline u32 fp_hash(struct fingerprint *fp) { - return fp->hash[0] ^ fp->hash[1] ^ fp->hash[2] ^ fp->hash[3]; + /* This hash is expected to be monotonic wrt. fpsort's order by fprecog */ + return (fp->hash[0] << 24) | (fp->hash[1] << 16) | (fp->hash[2] << 8) | fp->hash[3]; } /* URL keys */ -- 2.39.2