This is the mail archive of the
mailing list for the GCC project.
Re: using C++ STL containers in GCC/gfortran source code
- From: Pedro Alves <pedro at palves dot net>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: Janus Weil <janus at gcc dot gnu dot org>, gcc mailing list <gcc at gcc dot gnu dot org>, gfortran <fortran at gcc dot gnu dot org>
- Date: Fri, 16 Dec 2016 19:11:38 +0000
- Subject: Re: using C++ STL containers in GCC/gfortran source code
- Authentication-results: sourceware.org; auth=none
- References: <CAKwh3qgUN=mTFPg6Ew6Yk9RXgS3CttE7O3cGMQRcLKNqUOjBng@mail.gmail.com> <CAKwh3qii7EFG4VECNOgqF6kXczvwK0jQbQyDcQTnwuzSdSCfRw@mail.gmail.com> <20161216180415.GZ21933@tucnak> <firstname.lastname@example.org> <20161216185604.GD21933@tucnak>
On 12/16/2016 06:56 PM, Jakub Jelinek wrote:
> On Fri, Dec 16, 2016 at 06:52:03PM +0000, Pedro Alves wrote:
>> GDB has a string_printf function that prints into a std::string, for
>> example. Like:
>> std::string hello = string_printf ("%s", "hello world");
>> That's a function that many C++ projects reinvent.
> If you then want to work with it as std::string object, sure, it makes
> sense. But if all you want is to pass the string to %s of another
> formatting function and free it, then going through std::string
> doesn't add many benefits over just xasprintf + free.
It has all the usual advantages of RAII.
It completely eliminates the "forgot to call free in this
exit path" bug by design.
And then there's exception safety, in case something throws
beforeyou reach the "+ free".
(I know GCC doesn't use exceptions; GDB does. But do note
We would like the compiler to be exception safe, to permit
reconsideration of the exception convention. This change
would require a significant change in style,
adopting "resource acquisition is initialization" (RAII). We would be
using shared_ptr (from TR1's <memory>) or unique_ptr (from C++11).
We've started using RAII objects in GDB in the past couple
months, including C++11 unique_ptr, and that has already
simplified the codebase a good deal, and fixed many leaks