You are viewing a plain text version of this content. The canonical link for it is here.
Posted to c-dev@xerces.apache.org by "Roger Leigh (JIRA)" <xe...@xml.apache.org> on 2017/06/13 09:53:00 UTC
[jira] [Commented] (XERCESC-2100) [patch] Small fixes for warnings
and errors
[ https://issues.apache.org/jira/browse/XERCESC-2100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16047667#comment-16047667 ]
Roger Leigh commented on XERCESC-2100:
--------------------------------------
I've opened the ticket here in case anyone wanted to have a look over them and approve the changes. If there are no concerns with them, would it be OK to commit these on the trunk?
Thanks,
Roger
> [patch] Small fixes for warnings and errors
> -------------------------------------------
>
> Key: XERCESC-2100
> URL: https://issues.apache.org/jira/browse/XERCESC-2100
> Project: Xerces-C++
> Issue Type: Bug
> Affects Versions: 3.2.0
> Reporter: Roger Leigh
> Labels: patch
> Attachments: 0001-build-Merge-MsgCatalog-Makefile.in-with-src-Makefile.patch, 0002-cmake-Debug-FindThreads.patch, 0003-cmake-Minimum-C-standard-is-C-98-but-also-try-later-.patch, 0004-cmake-XercesIntTypes-Add-a-cstdint-functional-check.patch, 0005-cmake-Enable-extra-compiler-warnings.patch, 0006-samples-PSVIWriterHandlers.cpp-Use-delete-in-place-o.patch, 0007-tests-EncodingTest.cpp-Use-correct-format-strings-to.patch, 0008-tests-DTest.cpp-Use-correct-format-strings-to-match-.patch, 0009-tests-ThreadTest.cpp-Handle-all-node-types-in-switch.patch, 0010-tests-XSValueTest-Don-t-warn-about-integer-limit-pro.patch, 0011-xercesc-Don-t-warn-about-private-constructors-with-G.patch, 0012-xercesc-DOMParentNode-Add-missing-virtual-destructor.patch, 0013-xercesc-NamespaceScope-Add-missing-virtual-destructo.patch, 0014-xercesc-NamespaceScope-Correct-initialisation-order.patch, 0015-samples-PSVIWriterHandlers-Remove-unused-variables.patch, 0016-xercesc-DOMLSSerializerImpl-Remove-unused-variable.patch, 0017-xercesc-XMLUri-Remove-unused-variables.patch, 0018-xercesc-RangeToken-Conditionally-define-variable-if-.patch, 0019-xercesc-DatatypeValidatorFactory-Remove-unused-varia.patch, 0020-tests-DTest-Remove-unused-variables.patch, 0021-tests-MemoryMonitor-Remove-unused-variable.patch, 0022-tests-XSTSHarness-Remove-unused-variables.patch, 0023-tests-XSValueTest-Conditionally-define-conditionally.patch, 0024-xercesc-PlatformUtils-Include-sys-timeb.h-conditiona.patch, 0025-xercesc-BinMemInputStream-Add-missing-const_cast.patch, 0026-xercesc-QName-Add-mising-const_casts.patch, 0027-xercesc-XMLUri-Add-missing-const_cast.patch, 0028-xercesc-DOMLSSerializerImpl-Suppress-cast-alignment-.patch, 0029-xercesc-EncodingValidator-Suppress-cast-alignment-wa.patch, 0030-xercesc-XProtoType-Suppress-cast-alignment-warning.patch, 0031-xercesc-DOMCasts-Suppress-cast-alignment-warnings.patch, 0032-xercesc-XMLReader-Suppress-cast-alignment-warnings.patch, 0033-xercesc-XSerializeEngine-Suppress-cast-alignment-war.patch, 0034-xercesc-XML-Transcoder-Suppress-cast-alignment-warni.patch, 0035-xercesc-CMStateSet-Suppress-cast-alignment-warnings.patch
>
>
> These patches have been sitting around for nearly a year, but I've rebased them onto the trunk and tested them again. They cover several classes of fix:
> - minor build improvements
> - minor tweaks to feature tests
> - enabling stricter compiler warnings, and then fixing those warnings
> - fixing mismatched delete/delete[] (bad)
> - adding missing virtual destructors (bad)
> - removing unused variables
> - removing unused variables conditionally when used conditionally
> - removing cast warnings with appropriate C++ const/static/reinterpret casts
> Most of the fixes are tiny one-liners to fix warnings.
> Builds:
> - [travis|https://travis-ci.org/rleigh-codelibre/xerces-c/builds/241812965]
> - [appveyor|https://ci.appveyor.com/project/rleigh-codelibre/xerces-c/build/1.0.90]
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: c-dev-unsubscribe@xerces.apache.org
For additional commands, e-mail: c-dev-help@xerces.apache.org