bug-cvs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Question: windows-NT/config.h Change #define realloc rpl_realloc


From: Derek Robert Price
Subject: Re: Question: windows-NT/config.h Change #define realloc rpl_realloc
Date: Mon, 05 Apr 2004 13:03:07 -0400
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030624 Netscape/7.1

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Restarting my browser did the trick - here's the link to the thread:

http://groups.google.com/groups?hl=en&lr=&ie=UTF-8&oe=UTF-8&threadm=mailman.1025.1080246315.26040.bug-cvs%40gnu.org&rnum=1&prev=/groups%3Fq%3Dbohn%2Bderek%2Bstatus%2Bwindows%26hl%3Den%26lr%3D%26ie%3DUTF-8%26oe%3DUTF-8%26selm%3Dmailman.1025.1080246315.26040.bug-cvs%2540gnu.org%26rnum%3D1

Please blame Google for the long ugly link and not me.  :)

Derek

Derek Robert Price wrote:

> I've moved this discussion to bug-cvs.  This is the appropriate venue
> for development discussions.
>
> Conrad T. Pino wrote:
>
> Hi All,
>
> Being new to the project and coming from a Windoz background,
> I've been dabbling with building CVS with Visual C++ (VC) 6.0
> on the feature branch using the latest cvs.cvshome.org:/cvs
> *.dsp files.  I've been looking at the warnings generated in
> the compile which led to questions about recent changes made
> in the feature branch.
>
> I noticed "windows-NT/config.h" and it's precursors were updated
> to add "#define realloc rpl_realloc" which might be the root of a
> warning message I'm studying.
>
> Before coming to a conclusion about the warning, I'd like to get
> a better understanding of the reasons the change was made.  Can
> anyone shed some light here for me?
>
>
>
> I don't recall exactly, but I think some behavior the GNULIB xmalloc or
> xrealloc or something was expecting caused a crash on Windows.
> Something to do with what happened when a NULL pointer was passed in or
> somesuch.  Check the email archives for a thread between Chris Bohn & I
> discussing a patch he sent me.  My browser is on the fritz and won't let
> me get you a link, but the subject had "under Windows, cvs status
> reports" in it.
>
> - From src/ChangeLog:
>
> 2004-03-25  Derek Price  <derek@ximbiot.com>
>                                                                           
>  
>  
>  
>         * client.c (send_file_names): Initialize string to NULL rather
> than to
>         a single byte string containing only a null byte.
>         * subr.c (xrealloc_and_strcat): If the destination string is NULL,
>         simply allocate a new string.
>         (Original report from Chris Bohn <cbohn@rrinc.com>.)
>                                                                           
>  
>  
>  
>
> If you are really a glutton for punishment, you could probably check out
> the version from the 24th of March and reproduce the problem.
>
> Derek
> --
>                 *8^)
>
> Email: derek@ximbiot.com
>
> Get CVS support at <http://ximbiot.com>!


- --
                *8^)

Email: derek@ximbiot.com

Get CVS support at <http://ximbiot.com>!
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)
Comment: Using GnuPG with Netscape - http://enigmail.mozdev.org

iD8DBQFAcZFKLD1OTBfyMaQRAolHAJ9sweP2I9T5IvTRy1Owv+ypJ1V/nACg1J1d
zdtaYAgu5+DCgfZ5TTkA/hA=
=NCHp
-----END PGP SIGNATURE-----






reply via email to

[Prev in Thread] Current Thread [Next in Thread]