octave-maintainers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: eliminating or reducing use of "mutable" keyword?


From: Markus Mützel
Subject: Re: eliminating or reducing use of "mutable" keyword?
Date: Wed, 20 Nov 2019 10:36:24 +0100

Am 19. November 2019 um 19:32 Uhr schrieb "John W. Eaton":
> On 11/19/19 9:46 AM, "Markus Mützel" wrote:
> > Am 18. November 2019 um 23:20 Uhr schrieb "John W. Eaton":
> >> On 11/18/19 3:03 PM, Rik wrote:
> >>> jwe,
> >>>
> >>> I saw your recent changeset
> >>>
> >>> changeset:   27709:0a84a7d0998d
> >>> user:        John W. Eaton <address@hidden>
> >>> date:        Mon Nov 18 11:57:51 2019 -0500
> >>> summary:     don't declare figure::properties::toolkit as mutable
> >>>
> >>> which removes an instance of the mutable keyword.
> >>>
> >>> Should we make this a Task on Savannah?  The use of mutable seems very
> >>> occasionally necessary, but also very often a hack where it might be 
> >>> better
> >>> to re-think the code.
> >>>
> >>> I used grep and found 53 instances of the word in libinterp/ and 8 in
> >>> liboctave/.  The ones in libinterp are clustered in the MeX interface 
> >>> which
> >>> probably isn't worth scrubbing.
> >>
> >> Yes, I think we should aim to remove all uses of mutable if possible.
> >> In cases where it seems like it can't be removed, it would probably be
> >> worth documenting and justifying why it is really needed.
> >>
> > 
> > That change seems to have fixed the frequent segmentation faults of the 
> > Fedora buildbots, too.
> 
> Unfortunately, I'm not so sure that it is fixed.  I still see a segfault 
> when creating figures for the manual in this build:
> 
>    http://buildbot.octave.org:8010/#/builders/11/builds/1274

You're right. I spoke too soon.

Btw, the hg repo for the w64-64-stable buildbot seems to be corrupt. Could you 
please reset it?

Markus



reply via email to

[Prev in Thread] Current Thread [Next in Thread]