This is the mail archive of the
cygwin-patches
mailing list for the Cygwin project.
Re: [PATCH 09/11] Add c++14 sized deallocation operator
- From: Peter Foley <pefoley2 at pefoley dot com>
- To: cygwin-patches at cygwin dot com
- Date: Mon, 21 Mar 2016 15:34:38 -0400
- Subject: Re: [PATCH 09/11] Add c++14 sized deallocation operator
- Authentication-results: sourceware.org; auth=none
- References: <1458409557-13156-1-git-send-email-pefoley2 at pefoley dot com> <1458409557-13156-9-git-send-email-pefoley2 at pefoley dot com> <20160320112837 dot GO25241 at calimero dot vinschen dot de> <CAOFdcFPP79BaO=KTpF5oB3ewdYCh6GmfaxoJr03kKY7dSOjrKw at mail dot gmail dot com> <20160321171314 dot GA14892 at calimero dot vinschen dot de> <CAOFdcFM1D17HSiLdeNv=S6zim6wOcqY41Ud-iTtiDLrN_YRYOg at mail dot gmail dot com> <20160321180903 dot GB14892 at calimero dot vinschen dot de>
On Mon, Mar 21, 2016 at 2:09 PM, Corinna Vinschen
<corinna-cygwin@cygwin.com> wrote:
> I realized that your orignal patch isn't invalidated by this so I tried
> to apply it and we could then add the other stuff later. However, it
> doesn't compile due to a warning, and since we're always building with
> -Werror...
>
> [...]/cxx.cc:33:1: error: âvoid operator delete(void*, size_t)â is a usual (non-placement) deallocation function in C++14 (or with -fsized-deallocation) [-Werror=c++14-compat]
> operator delete (void *p, size_t)
> ^
> cc1plus: all warnings being treated as errors
>
> I'm not sure it's the right thing to switch to C++14 by default using
> gcc 5.3 yet.
Ah, in that case, a better solution might be to drop this patch and
add an explicit -std= to the Makefile.
In that case, Cygwin won't have any issues when the default changes to
c++14 in gcc 6.0
I'm not sure what level of -std would be appropriate by default though.
Thanks,
Peter Foley