comp.lang.ada
 help / color / mirror / Atom feed
From: "Dmitry A. Kazakov" <mailbox@dmitry-kazakov.de>
Subject: Re: Generic Embedded List Nodes
Date: Mon, 20 Jun 2016 09:25:16 +0200
Date: 2016-06-20T09:25:16+02:00	[thread overview]
Message-ID: <nk85pq$1qpj$1@gioia.aioe.org> (raw)
In-Reply-To: fcef381c-b0c0-4c53-9b99-012b7442a532@googlegroups.com

On 20/06/2016 04:42, Warren wrote:
> On Sunday, 19 June 2016 16:35:32 UTC-4, Dmitry A. Kazakov  wrote:
>> No, doubly-linked list deletion is O(1).
>
> Ok you're tracking the link in Element, which is fine. However, your
> Element also needs a reference to the separately allocated object (which
> is a problem for me). This requires two allocations instead of one.

No, that is the point. The object and the element (links) are in one 
continuous chunk of memory. I believe this is what you meant under being 
"embedded" nodes.

When an object is allocated, element + object is instead and the address 
to the object's part is returned back from Allocate. Deallocate takes 
the object's address, subtracts the offset and deallocates the whole chunk.

> I only need to insert head, traversal and delete. That's it!

Yes, and the schema above is as effective as it can be.

>>> In the embedded node case, I already have direct access to the
>>> affected link node. To remove the node from a list I simply say:
>>>
>>>     R.Link_Node.Unlink;
>>
>> The operation Delete has the list head parameter (Container) not for
>> traversing the list, but for modifying the list head if the first
>> element is deleted from the list.
>
>> If you don't have it, you must maintain a dedicated list head element
>> with no object attached. That is a less safe and clean because it
>> ultimately leads to run-time type checks in the client code.
>
> I agree with the dedicated list head statement, but not the "less
> safe" part. You either have container or you have a list head (each
> represents one list, though yours potentially several).
>
> There is nothing to check about a list head- you simply begin there.
> If you have no "head.next", you have an empty list.

Not with doubly-linked lists. There is always Next, because the list is 
circular. When you delete an element from the list you always get two 
lists. Deletion of a single element is an idempotent operation unless 
you have a dedicated head or else have the list head pointer corrected.

List traversal when the list head is a pointer is performed like this:

    if Head /= null then
       This := Head;
       loop
          ... -- Do something
          This := This.Next;
          exit when This = Head;
       end loop;
    end if;

-- 
Regards,
Dmitry A. Kazakov
http://www.dmitry-kazakov.de

  reply	other threads:[~2016-06-20  7:25 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-18 22:52 Generic Embedded List Nodes Warren
2016-06-18 23:40 ` Jeffrey R. Carter
2016-06-19  2:15   ` Warren
2016-06-19  3:04     ` Jeffrey R. Carter
2016-06-19  2:14 ` Jeremiah
2016-06-19  2:21   ` Warren
2016-06-19  2:50     ` Warren
2016-06-19  4:45       ` Simon Wright
2016-06-19 18:27         ` Warren
2016-06-19 19:04           ` Dmitry A. Kazakov
2016-06-19 20:13             ` Warren
2016-06-19 20:35               ` Dmitry A. Kazakov
2016-06-20  2:42                 ` Warren
2016-06-20  7:25                   ` Dmitry A. Kazakov [this message]
2016-06-20 12:26                     ` Warren
2016-06-20 19:33                       ` Niklas Holsti
2016-06-21  2:20                         ` Warren
2016-06-21  5:52                           ` Niklas Holsti
2016-06-21  7:15                             ` Dmitry A. Kazakov
2016-06-21 18:54                               ` Niklas Holsti
2016-06-21 19:54                                 ` Dmitry A. Kazakov
2016-06-21 10:31                             ` Warren
2016-06-21 17:13                               ` Jeffrey R. Carter
2016-06-21 18:56                                 ` Niklas Holsti
2016-06-21 20:13                                   ` Warren
2016-06-21 21:38                               ` Niklas Holsti
2016-06-23  2:12                                 ` Warren
2016-06-23  8:19                                   ` Niklas Holsti
2016-06-23 12:37                                     ` Warren
2016-06-23 15:36                                       ` Niklas Holsti
2016-06-24  1:55                                         ` Warren
2016-06-24 12:49                                         ` Warren
2016-06-25  5:50                                           ` Niklas Holsti
2016-06-26  1:36                                             ` Warren
2016-07-01 13:49                                             ` Warren
2016-07-01 16:28                                               ` Warren
2016-06-24 20:25                                         ` Warren
2016-06-22 13:01                               ` G.B.
2016-06-23  2:30                                 ` Warren
2016-06-20  6:08 ` Niklas Holsti
2016-06-20 12:20   ` Warren
2016-06-20 19:47 ` Shark8
2016-06-21  2:28   ` Warren
2016-06-21  7:21     ` Dmitry A. Kazakov
2016-06-21 10:32       ` Warren
2016-06-21 11:56         ` Dmitry A. Kazakov
2016-06-21 13:39           ` Warren
2016-06-21 14:04             ` Dmitry A. Kazakov
2016-06-23  0:37     ` Randy Brukardt
2016-06-23  2:25       ` Warren
2016-07-01 19:50 ` brbarkstrom
2016-07-02  1:55   ` Warren
replies disabled

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox