GNAT already emits a style warning when redundant parentheses appear inside
logical and short-circuit operators; this warning will be soon emitted for
unary operators as well.
There are two similar generic procedures in POSIX.Process_Environment:
* For_Every_Environment_Variable (Env : Environment)
* For_Every_Current_Environment_Variable
The one with explicit parameter adjusted the bounds of the Value string
that is passed to generic actual subprogram; now the one with implicit
parameter does the same adjustment.
Previous fix was incorrect, 'not' expression must keep its parentheses.
And fix extra paren in "<" function.
no-precommit-check (to avoid complains about copyright headers)
TN: W203-029
Change-Id: I8102f1c086d35c984eca7e50e1570a05a5f33f65
New warning has been enabled in the frontend. This change fixes one
new occurence of this new warning in debug build.
no-precommit-check (to avoid complains about copyright headers)
TN: W203-029
Change-Id: I0c5be7b9bebc2a6cd11c027831a5317c722ded19
New warning has been enabled in the frontend. This change fixes the 4
new occurences of this new warning.
no-precommit-check (to avoid complains about copyright headers)
TN: W203-029
Change-Id: I5d36920109557a6337e8db7d6eaacc6a70bccc3a
Change uses of Unchecked_Conversion (which is obsolete since Ada 95) to
Ada.Unchecked_Conversion to avoid warnings recently enabled in GNAT. Same
for Unchecked_Deallocation.
Previous patch only changed files used for production build; this patch
changes files used for debug builds (and also for gnatsocks, for
completeness).
no-precommit-check (to avoid complains about copyright headers)
Change-Id: I0a753b06a46ce56778b8de9c96607515bea7cf84
Change uses of Unchecked_Conversion (which is obsolete since Ada 95) to
Ada.Unchecked_Conversion to avoid warnings recently enabled in GNAT.
no-precommit-check (to avoid complains about copyright headers)
TN: V209-051
Change-Id: I62d8385a92eb200c0434d440df02712c28554d2b
Expose O_CLOEXEC, O_DIRECTORY, and O_NOFOLLOW flags to API.
O_EXEC, O_SEARCH, and O_TTY_INIT are not exposed because not supported
on Linux.
TN: SC04-017
Change-Id: I43ee1270640294049bc96877117bc6372c63c5e4
This fixes the following warnings:
warning: unnecessary with of ancestor [-gnatwr]
_gprbuild: *** compilation phase failed
no-precommit-check
TN: S506-002
Change-Id: I206849c43821380eed71a32688335a993ee5d76a
Remove unnecessary "with POSIX", which the compiler now
warns about.
no-precommit-check
TN: UB17-024
Change-Id: I8f4ac3f629519fe5bb7222c8dcbe0a1bb2e3db7c
Remove unnecessary "with POSIX", which the compiler now
warns about.
no-precommit-check
TN: UB17-024
Change-Id: I117c0f72947b5bfbf72f58c212d1da9a3abc4be7
While compiling with --with-build-type=Debug:
posix.adb:454:07: warning: use clause for package "Bogus_Error_Codes"
has no effect
gprbuild: *** compilation phase failed
TN: S503-023
SIGRTMIN macro is not defined under Darwin and __gnat_florist_sigrtmin is
not used either on this platform so don't try to compile it in
posix-macros.c.
Tested under Darwin and Linux.
TN: S217-004
The SIGRTMIN constant was hardcoded to the SIGRTMIN C macro value at
compilation time. Unfortunately this is wrong as it can change at
runtime according to the manual.
Excerpt from bits/signum.h:
"#define SIGRTMIN (__libc_current_sigrtmin ())"
So drop the SIGRTMIN constant for a C function wrapper
(__gnat_florist_sigrtmin) which calls the SIGRTMIN macro in the C land.
no-precommit-check
TN: S217-004
Change POSIX_Character so it's a subtype,
so that type conversions between String
and POSIX_String work. Such conversions are
a lot more efficient than calls to the
conversion functions.
R426-014
