[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: unicode build: "comparison is always true" warnings
From: |
Mark A. Hershberger |
Subject: |
Re: unicode build: "comparison is always true" warnings |
Date: |
Fri, 23 Nov 2007 13:42:38 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.0.60 (gnu/linux) |
Andreas Schwab <address@hidden> writes:
> address@hidden (Mark A. Hershberger) writes:
>
>> while (1)
>> {
>> /* Create the lock file, but not under the lock file name. */
>> /* Give up if cannot do that. */
>> +#ifndef HAVE_MKSTEMP
>> desc = open (tempname, O_WRONLY | O_CREAT | O_EXCL, 0666);
>> +#else
>> + desc = mkstemp (tempname);
>> +#endif
>
> That won't work since mkstemp modifies tempname in place, but can be
> called more than once here (the second call will return with EINVAL).
There is only one call.
If you look at the full patch, mktemp (not mkstemp) is called once if
HAVE_MKSTEMP is _not_ defined. Here, mkstemp (not mktemp) is called if
HAVE_MKSTEMP _is_ defined.
Mark.
--
http://hexmode.com/
GPG Fingerprint: 7E15 362D A32C DFAB E4D2 B37A 735E F10A 2DFC BFF5
The most beautiful experience we can have is the mysterious.
-- Albert Einstein, The World As I See it
- Re: unicode build: "comparison is always true" warnings, (continued)
- Re: unicode build: "comparison is always true" warnings, Matthieu Lemerre, 2007/11/24
- Re: unicode build: "comparison is always true" warnings, Andreas Schwab, 2007/11/24
- Re: unicode build: "comparison is always true" warnings, Matthieu Lemerre, 2007/11/24
- Re: unicode build: "comparison is always true" warnings, David Kastrup, 2007/11/24
- Re: unicode build: "comparison is always true" warnings, Andreas Schwab, 2007/11/24
- Re: unicode build: "comparison is always true" warnings, David Kastrup, 2007/11/24
- Re: unicode build: "comparison is always true" warnings,
Mark A. Hershberger <=
- Re: unicode build: "comparison is always true" warnings, Andreas Schwab, 2007/11/23