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: John W. Eaton
Subject: Re: eliminating or reducing use of "mutable" keyword?
Date: Tue, 19 Nov 2019 12:32:52 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0

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

jwe



reply via email to

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