Thread overview
[Issue 21250] dirEntries on non-existant directory causes assert error
[Issue 21250] dirEntries on non-existent directory causes assert error
Sep 30
Dlang Bot
Oct 02
Dlang Bot
Oct 18
Dlang Bot
September 15
https://issues.dlang.org/show_bug.cgi?id=21250

John Colvin <john.loughran.colvin@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |industry
           Severity|enhancement                 |major

--
September 16
https://issues.dlang.org/show_bug.cgi?id=21250

Stefan Koch <uplink.coder@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |uplink.coder@gmail.com

--- Comment #1 from Stefan Koch <uplink.coder@gmail.com> ---
What happens here is a ctor dtor mismatch.

A temporary of a Refcounted is created inside a function where the file not
found exception is thrown.
The dtor is called on that temporary, which is basically a ref of the
refcounted since the refcounted has ref semantics on the count.
when the execption passes up a level the destructor of the actual refcouted
value is called. which tries to decrement the refcount again.

--
September 16
https://issues.dlang.org/show_bug.cgi?id=21250

--- Comment #2 from Stefan Koch <uplink.coder@gmail.com> ---
Related: https://issues.dlang.org/show_bug.cgi?id=21036

--
September 17
https://issues.dlang.org/show_bug.cgi?id=21250

Atila Neves <atila.neves@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |atila.neves@gmail.com
            Summary|dirEntries on non-existant  |dirEntries on non-existent
                   |directory causes assert     |directory causes assert
                   |error                       |error

--
September 17
https://issues.dlang.org/show_bug.cgi?id=21250

John Colvin <john.loughran.colvin@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|major                       |regression

--
September 30
https://issues.dlang.org/show_bug.cgi?id=21250

Andrei Alexandrescu <andrei@erdani.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |andrei@erdani.com

--- Comment #3 from Andrei Alexandrescu <andrei@erdani.com> ---
Possibly related: https://github.com/dlang/phobos/pull/7645

--
September 30
https://issues.dlang.org/show_bug.cgi?id=21250

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #4 from Dlang Bot <dlang-bot@dlang.rocks> ---
@andralex created dlang/phobos pull request #7646 "Andralex patch 3" fixing this issue:

- Fix Issue 21250 - dirEntries on non-existent directory causes assert error

https://github.com/dlang/phobos/pull/7646

--
October 02
https://issues.dlang.org/show_bug.cgi?id=21250

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/phobos pull request #7646 "Issue 21250 - dirEntries on non-existent directory causes assert error" was merged into stable:

- b9e20a58953950e92fb19a397f77fa90e1461395 by Andrei Alexandrescu:
  Fix Issue 21250 - dirEntries on non-existent directory causes assert error

https://github.com/dlang/phobos/pull/7646

--
October 18
https://issues.dlang.org/show_bug.cgi?id=21250

--- Comment #6 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/phobos pull request #7664 "merge stable" was merged into master:

- c5e6213dde8fc0949c39ec2364abfe2070a67b2b by Andrei Alexandrescu:
  Issue 21250 - dirEntries on non-existent directory causes assert error
(#7646)

  Issue 21250 - dirEntries on non-existent directory causes assert error
  merged-on-behalf-of: Andrei Alexandrescu <andralex@users.noreply.github.com>

https://github.com/dlang/phobos/pull/7664

--