Greetings,
Well spotted. It shouldn't matter, though. The comments talk about
"default" page size, so it could well be overridden anyway... Notice
that it is set to at most 16k a few lines above, so it isn't likely
to cause a 64k limit.
Cheers,
Lachlan
On Thu, 19 Jun 2003 04:00, Neal Richter wrote:
> Hey,
> I think I have found something here.. no idea of importance yet.
>
> In db/db.c line 1049 we have an #ifdef:
>
> /*
> * If compression is on, the minimum page size must be
> multiplied * by the compression factor.
> */
> #ifdef HAVE_ZLIB
> if(F_ISSET(dbp, DB_AM_CMPR)) {
> if(iopsize < DB_CMPR_MULTIPLY(dbenv, DB_MIN_PGSIZE))
> iopsize = DB_CMPR_MULTIPLY(dbenv, DB_MIN_PGSIZE);
> }
> #endif /* HAVE_ZLIB */
>
>
> This was added to db.c 3 years ago when Geoff merged Loic's code
> from milfuz.
>
> The problem is that it needs to be HAVE_LIBZ rather than HAVE_ZLIB!
> Our db_config.h contains HAVE_LIBZ & HAVE_ZLIB_H .. no HAVE_ZLIB.
>
> I am not sure yet how this affects us yet... it looks like a range
> check and may be part of why db-compression doesn't work when I try
> and set the wordlist_page_size to 64K.
>
> I also checked and this is the only occurance of this screwup.
>
> Obviously it must not be too important since we do have functional
> libz-based DB compression.
>
> Thanks.
>
> Neal Richter
> Knowledgebase Developer
> RightNow Technologies, Inc.
> Customer Service for Every Web Site
> Office: 406-522-1485
--
[EMAIL PROTECTED]
ht://Dig developer DownUnder (http://www.htdig.org)
-------------------------------------------------------
This SF.Net email is sponsored by: INetU
Attention Web Developers & Consultants: Become An INetU Hosting Partner.
Refer Dedicated Servers. We Manage Them. You Get 10% Monthly Commission!
INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php
_______________________________________________
htdig-dev mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/htdig-dev
