* [ECOS] string functions assertions
@ 2001-08-09 3:07 Robin Farine
2001-08-10 10:24 ` Jonathan Larmour
0 siblings, 1 reply; 2+ messages in thread
From: Robin Farine @ 2001-08-09 3:07 UTC (permalink / raw)
To: ecos-discuss
Hi,
The string functions defined in "packages/language/c/libc/string/<ver>/src" that
take a length argument such as memmove(...) fail when called with a length of 0
if compiled with CYGPKG_INFRA_DEBUG enabled. I suppose that it's legal to do
that so I added some length tests to those functions (these tests should get
optimized out when compiling with assertions disabled). The attached patch
contains the changes.
Robin
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [ECOS] string functions assertions
2001-08-09 3:07 [ECOS] string functions assertions Robin Farine
@ 2001-08-10 10:24 ` Jonathan Larmour
0 siblings, 0 replies; 2+ messages in thread
From: Jonathan Larmour @ 2001-08-10 10:24 UTC (permalink / raw)
To: Robin Farine; +Cc: ecos-discuss
Robin Farine wrote:
>
> Hi,
>
> The string functions defined in "packages/language/c/libc/string/<ver>/src" that
> take a length argument such as memmove(...) fail when called with a length of 0
> if compiled with CYGPKG_INFRA_DEBUG enabled. I suppose that it's legal to do
> that so I added some length tests to those functions (these tests should get
> optimized out when compiling with assertions disabled). The attached patch
> contains the changes.
Yep, fine. And your stdio read patch too. I've checked these in. I'll write
ChangeLog entries for these this time, but you can still write the cvt()
changelog :-).
Jifl
--
Red Hat, Rustat House, Clifton Road, Cambridge, UK. Tel: +44 (1223) 271062
Maybe this world is another planet's Hell -Aldous Huxley || Opinions==mine
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2001-08-10 10:24 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-08-09 3:07 [ECOS] string functions assertions Robin Farine
2001-08-10 10:24 ` Jonathan Larmour
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).