[Issue 11216] Make synchronized statement `nothrow`

2014-12-12 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

--- Comment #5 from github-bugzi...@puremagic.com ---
Commits pushed to master at https://github.com/D-Programming-Language/druntime

https://github.com/D-Programming-Language/druntime/commit/05b15ab05279a70b30f4a580c705526e70e09155
fix Issue 11216 - Make synchronized statement `nothrow`

- require Object.Monitor lock and unlock functions to be nothrow

- change methods of all core.sync classes to nothrow

https://github.com/D-Programming-Language/druntime/commit/46061e646ae547d297aeae448eb246e251118344
Merge pull request #1013 from MartinNowak/fix11216

fix Issue 11216 - Make synchronized statement `nothrow`

--


[Issue 11216] Make synchronized statement `nothrow`

2015-02-18 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

--- Comment #6 from github-bugzi...@puremagic.com ---
Commits pushed to 2.067 at https://github.com/D-Programming-Language/druntime

https://github.com/D-Programming-Language/druntime/commit/05b15ab05279a70b30f4a580c705526e70e09155
fix Issue 11216 - Make synchronized statement `nothrow`

https://github.com/D-Programming-Language/druntime/commit/46061e646ae547d297aeae448eb246e251118344
Merge pull request #1013 from MartinNowak/fix11216

--


[Issue 11216] Make synchronized statement `nothrow`

2015-02-18 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

--- Comment #7 from github-bugzi...@puremagic.com ---
Commits pushed to 2.067 at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/47c679964003ca75290e3174df85b0641eb5d282
fix Issue 11216 - Make synchronized statement `nothrow`

https://github.com/D-Programming-Language/dmd/commit/dc118391e51ddb69093780c157b1ca41e26add90
Merge pull request #4115 from MartinNowak/fix11216

--


[Issue 11216] Make synchronized statement `nothrow`

2015-10-18 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

Mathias LANG  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 CC||pro.mathias.l...@gmail.com
 Resolution|FIXED   |---

--- Comment #8 from Mathias LANG  ---
Well, since it was reverted, it should have been reopened.

Revert in DMD: https://github.com/D-Programming-Language/dmd/pull/4459
Revert in Druntime:
https://github.com/D-Programming-Language/druntime/pull/1188

--


[Issue 11216] Make synchronized statement `nothrow`

2016-05-10 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

Mathias Lang  changed:

   What|Removed |Added

 CC||thecybersha...@gmail.com

--- Comment #9 from Mathias Lang  ---
*** Issue 16009 has been marked as a duplicate of this issue. ***

--


[Issue 11216] Make synchronized statement `nothrow`

2018-02-23 Thread d-bugmail--- via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

FeepingCreature  changed:

   What|Removed |Added

 CC||default_357-l...@yahoo.de

--- Comment #10 from FeepingCreature  ---
Still an issue in 2018.

--


[Issue 11216] Make synchronized statement `nothrow`

2018-02-23 Thread d-bugmail--- via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

--- Comment #11 from FeepingCreature  ---
Istm a possible approach would be:

 * Define SimpleMonitor : Monitor with nothrow

 * change DMD to use nothrow when locking SimpleMonitors

 * subclass druntime's Mutex with SimpleMutex : Mutex, SimpleMonitor

 * use SimpleMutex as the mutex implementation for class monitors

Then synchronized(this) should be nothrow without breaking existing code that
depends on Mutex/Monitor. It would only break if you were assigning to
_monitor, and that's easily fixed by using a separate member.

I don't know how to implement this on the DMD side though.

--


[Issue 11216] Make synchronized statement `nothrow`

2013-10-10 Thread d-bugmail
http://d.puremagic.com/issues/show_bug.cgi?id=11216



--- Comment #1 from Denis Shelomovskij  2013-10-10 
17:14:31 MSD ---
Also in the case in the description are there any possibilities to supply a
user monitor? If not, this subcase is clearly a "rejects-valid" bug.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---


[Issue 11216] Make synchronized statement `nothrow`

2014-10-01 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

Martin Nowak  changed:

   What|Removed |Added

   Priority|P2  |P5
 CC||c...@dawg.eu

--- Comment #2 from Martin Nowak  ---
I'm in favor of doing this. Is there a migration path for requiring nothrow on
the Monitor Interface without breaking code?

--


[Issue 11216] Make synchronized statement `nothrow`

2014-10-01 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

Martin Nowak  changed:

   What|Removed |Added

 Blocks||13561

--


[Issue 11216] Make synchronized statement `nothrow`

2014-11-07 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

Walter Bright  changed:

   What|Removed |Added

 CC||bugzi...@digitalmars.com

--- Comment #3 from Walter Bright  ---
https://github.com/D-Programming-Language/dmd/pull/4115

--


[Issue 11216] Make synchronized statement `nothrow`

2014-11-08 Thread via Digitalmars-d-bugs
https://issues.dlang.org/show_bug.cgi?id=11216

--- Comment #4 from github-bugzi...@puremagic.com ---
Commits pushed to master at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/47c679964003ca75290e3174df85b0641eb5d282
fix Issue 11216 - Make synchronized statement `nothrow`

- mark _d_monitorenter/exit as nothrow

https://github.com/D-Programming-Language/dmd/commit/dc118391e51ddb69093780c157b1ca41e26add90
Merge pull request #4115 from MartinNowak/fix11216

fix Issue 11216 - Make synchronized statement `nothrow`

--