diff options
author | 2018-03-31 11:59:20 -0500 | |
---|---|---|
committer | 2018-03-31 11:59:20 -0500 | |
commit | 4057859ec03ac04a31eb28d2275961f6019c8ae1 (patch) | |
tree | 504ba916143f104edff32150372149831596a2b8 /ChangeLog.txt | |
parent | f0fa9fba3c53bf4f87f33b47a583cb031b962d34 (diff) | |
download | pseudo-4057859ec03ac04a31eb28d2275961f6019c8ae1.tar.gz pseudo-4057859ec03ac04a31eb28d2275961f6019c8ae1.tar.bz2 pseudo-4057859ec03ac04a31eb28d2275961f6019c8ae1.zip |
don't change errno in pseudo_fix_path
Two issues: (1) pseudo_fix_path could call lstat("") which generated
ENOENT, (2) some people, and I'm totally not staring intently
at cross-localedef here, don't know that it is in general *not*
valid to check errno unless an operation has failed, and the mere
fact that something set errno does not mean that the error indicated
actually occurred or has any relevance to you.
Nonetheless, pseudo_fix_path now saves and restores errno,
and cross-localedef can just go ahead and check it as though it
were guaranteed not to sometimes get set even though no error
occurred.
Signed-off-by: Seebs <seebs@seebs.net>
Diffstat (limited to 'ChangeLog.txt')
-rw-r--r-- | ChangeLog.txt | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/ChangeLog.txt b/ChangeLog.txt index e757596..f984820 100644 --- a/ChangeLog.txt +++ b/ChangeLog.txt @@ -1,3 +1,10 @@ +2018-03-31: + * (seebs) keep pseudo_fix_path from changing errno, because + *some* people think they should check errno to find out + whether an error occurred, even if no call has returned + a status indicating an error, and this is completely wrong + but we can't fix them all. + 2018-03-30: * (seebs) fix path stuff more thoroughly. * (seebs) Merge suggested README change from Richard Tollerton |