[phobos] [D-Programming-Language/phobos] ce17ab: fix, file handle must not be visible.

GitHub via phobos phobos at puremagic.com
Tue Apr 28 18:01:33 PDT 2015


  Branch: refs/heads/master
  Home:   https://github.com/D-Programming-Language/phobos
  Commit: ce17abef8759a261b656c27db8dbd4d3dca8dfc2
      https://github.com/D-Programming-Language/phobos/commit/ce17abef8759a261b656c27db8dbd4d3dca8dfc2
  Author: Basile Burg <basile.burg at gmx.com>
  Date:   2015-04-27 (Mon, 27 Apr 2015)

  Changed paths:
    M std/cstream.d

  Log Message:
  -----------
  fix, file handle must not be visible.

to change directly the handle could lead to leak the handle in `close()` because `isopen` was not defined
like it's the case when the `file()` setter is properly called.

possible breakage: no property...for class...
CFile.file() must be called instead.


  Commit: ddeacd73a5382e805d691ab8eae19c755599e6fa
      https://github.com/D-Programming-Language/phobos/commit/ddeacd73a5382e805d691ab8eae19c755599e6fa
  Author: Andrei Alexandrescu <andrei at erdani.com>
  Date:   2015-04-28 (Tue, 28 Apr 2015)

  Changed paths:
    M std/cstream.d

  Log Message:
  -----------
  Merge pull request #3237 from BBasile/fix-cstream-prot

fix, file handle must not be visible.


Compare: https://github.com/D-Programming-Language/phobos/compare/ab5601e5638e...ddeacd73a538


More information about the phobos mailing list