Yah, the title doesn't reflect it. But a DList is an obvious continuation of std.container and I just didn't care about it being a separate pull request. The design of DList is fairly fixed, there's little need to e.g. have a formal review for it.

Of course adding allocators will affect DList and all other containers, but I hope in a rather uniform way.


Andrei

On 7/9/12 12:33 AM, Masahiro Nakagawa wrote:
I was a little surprised.
I didn't know DList proposal until after I read this mail.
Why did this pull request mix with fixing and adding new feature?


On Mon, Jul 9, 2012 at 12:07 PM, GitHub<[email protected]>  wrote:
   Branch: refs/heads/master
   Home:   https://github.com/D-Programming-Language/phobos
   Commit: 64c5e3cffa58fecdb8ecb23fd175e5b8dcede2a7
       
https://github.com/D-Programming-Language/phobos/commit/64c5e3cffa58fecdb8ecb23fd175e5b8dcede2a7
   Author: Revellian<[email protected]>
   Date:   2012-06-25 (Mon, 25 Jun 2012)

   Changed paths:
     M std/container.d

   Log Message:
   -----------
   Fix std.container.SList.linearRemove's TakeRange-bug (if the TakeRange 
starts at the
head of the SList, the whole list was removed instead of only the
specified number.)


   Commit: 0cae7ca6af24f6ad322edbe263f43f88b5c5650a
       
https://github.com/D-Programming-Language/phobos/commit/0cae7ca6af24f6ad322edbe263f43f88b5c5650a
   Author: Revellian<[email protected]>
   Date:   2012-06-25 (Mon, 25 Jun 2012)

   Changed paths:
     M std/container.d

   Log Message:
   -----------
   Added doubly-linked list (DList) to std.container


   Commit: 61fa7c809cd4db2e214f5f2702690bd52436b7a0
       
https://github.com/D-Programming-Language/phobos/commit/61fa7c809cd4db2e214f5f2702690bd52436b7a0
   Author: Revellian<[email protected]>
   Date:   2012-07-02 (Mon, 02 Jul 2012)

   Changed paths:
     M std/container.d

   Log Message:
   -----------
   - Incorporate comments by andralex


   Commit: e0757c5ded416d7c781a8199ae2f73eab7da3dc6
       
https://github.com/D-Programming-Language/phobos/commit/e0757c5ded416d7c781a8199ae2f73eab7da3dc6
   Author: Andrei Alexandrescu<[email protected]>
   Date:   2012-07-08 (Sun, 08 Jul 2012)

   Changed paths:
     M std/container.d

   Log Message:
   -----------
   Merge pull request #650 from revellian/master

Fix std.container.SList.linearRemove's TakeRange-bug


Compare: 
https://github.com/D-Programming-Language/phobos/compare/865c7a6f9d00...e0757c5ded41

_______________________________________________
phobos mailing list
[email protected]
http://lists.puremagic.com/mailman/listinfo/phobos
_______________________________________________
phobos mailing list
[email protected]
http://lists.puremagic.com/mailman/listinfo/phobos
_______________________________________________
phobos mailing list
[email protected]
http://lists.puremagic.com/mailman/listinfo/phobos

Reply via email to