This is the mail archive of the libstdc++@gcc.gnu.org mailing list for the libstdc++ 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]

Current analysis of libstdc++-v3 testsuite run against -pedantic


I wrote:

> [ext/rope.cc actually finds the bug when testsuite run with -pedantic
> flag, thus no new test case deemed required IMHO.  Instead, we should
> consider enabling a pass of the library testsuite with that flag OR
> just encourage us developers to run with that flag every now and
> then.  For good measure, I will do so on mainline.]

Here is the current report with commentary:

$ RUNTESTFLAGS='--target_board ''unix{-pedantic}'''
$ gmake -sk check
[...output filtered to display new FAIL only...]
FAIL: 21_strings/char_traits_requirements.cc (test for excess errors)
FAIL: 22_locale/ctype_is_char.cc (test for excess errors)
FAIL: 22_locale/ctype_to_char.cc (test for excess errors)
FAIL: 22_locale/num_get_members_char.cc (test for excess errors)
FAIL: 22_locale/num_put_members_char.cc (test for excess errors)
FAIL: 23_containers/list_operators.cc (test for excess errors)
FAIL: 27_io/filebuf_virtuals.cc (test for excess errors)
FAIL: 27_io/istream_extractor_char.cc (test for excess errors)
FAIL: 27_io/istream_unformatted.cc (test for excess errors)
FAIL: 27_io/streambuf_members.cc (test for excess errors)

	All of the above fail due to something non-pedantic-compliant in
	the actual test cases (i.e. not so important to fix to avoid
	future PRs):

	ISO C++ forbids variable-size array `X'
	ISO C++ does not support `long long' [and related errors]
	ISO C++ forbids declaration of `main' with no type

FAIL: backward/header_slist_h.cc (test for excess errors)
FAIL: ext/rope.cc (test for excess errors)

	The last two fail due to issues within the library
	code/headers itself (but which are correctly masked once the
	headers are found in their installed location; and,
	interestingly enough, can't seem to be provoked by adding even
	``-pedantic -Wsystem-headers''):

	include/bits/stl_alloc.h:968: warning:
	ISO C++ forbids the use of `extern' on explicit instantiations
	include/bits/stl_alloc.h:973: warning:
	ISO C++ forbids the use of `extern' on explicit instantiations

In sum, no current extra real failures with -pedantic, but it would
take some work to enable as an extra pass at the moment.

FYI, ext/rope.cc did indeed fail with additional errors before the
patch for libstdc++/7922.

Regards,
Loren


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