Re: current CVS: undefined reference to `PGLZ_RAW_SIZE'

From: JanWieck(at)t-online(dot)de (Jan Wieck)
To: PostgreSQL HACKERS <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: current CVS: undefined reference to `PGLZ_RAW_SIZE'
Date: 2000-07-05 10:06:49
Message-ID: 200007051006.MAA09154@hot.jw.home
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane wrote:
> Thomas Lockhart <lockhart(at)alumni(dot)caltech(dot)edu> writes:
> >> utils/SUBSYS.o: In function `lztextlen':
> >> utils/SUBSYS.o(.text+0x11c68): undefined reference to `PGLZ_RAW_SIZE'
>
> > Did you do this from a completely fresh tree? If not, did you do a "make
> > clean" first? I built this morning without trouble...
>
> No, he's right: the lztext stuff has an undefined reference that's
> buried inside #ifdef MULTIBYTE. Not sure if the answer is to add
> the missing #include to lztext.c, or if the code is wrong. Jan?

To me it looks more that the code is wrong, left over from
the old self compressing version of lztext. I think it should
be strlen(s1) instead.

I changed it to that. Could you please check if it works?

Jan

--

#======================================================================#
# It's easier to get forgiveness for being wrong than for being right. #
# Let's break this rule - forgive me. #
#================================================== JanWieck(at)Yahoo(dot)com #

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Sevo Stille 2000-07-05 10:38:25 Re: Repair plan for inet and cidr types
Previous Message Chris Bitmead 2000-07-05 09:59:13 Re: Proposed new libpq API