This is the mail archive of the mailing list for the glibc project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Make pthread_getspecific async-signal-safe

I think we might be open to declaring pthread_getspecific async-signal-safe
as a GNU extension.  If we do this, we need to make the manual say very
explicitly that we're offering this guarantee as part of the glibc API even
though it is not guaranteed by POSIX.  So please include such wording
changes to manual/threads.texi in your patch.

The code change itself needs thorough comments explaining what constraints
the code is (newly) under and exactly what, why, and how it is doing to
meet them.

The test program has some common style errors: missing descriptive comment
in top line; two spaces after a period in comments; space before paren in
function calls; space after cast.

It's also generally bad form to write code so important side effects are in
assert, even though it's not a semantic error in the context of libc tests
since we'll never build them with -DNDEBUG.

The test could use some comments about the logic of what it's testing.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]