Namespace name of PartitionAlloc standalone library

14 views
Skip to first unread message

Yuki Shiino

unread,
Nov 16, 2021, 8:43:31 AM11/16/21
to memor...@chromium.org, Takashi Sakamoto, li...@chromium.org, bar...@chromium.org
Hi memory-dev@,

We're now working to make PartitionAlloc library a standalone library (a separate library from //base), and then we need a new namespace which is unique to the new PartitionAlloc library.  What namespace name would you recommend?

The current candidates are:
1. partition_allocator
2. partition_alloc
3. pa (acronym of Partition Allocator)

For example, the current ::base::PartitionRoot will be renamed to ::partition_allocator::PartitionRoot.

Other ideas are very welcome, and let us know your recommendations.  We'd like to make an agreement on the namespace name on this thread.

Cheers,
Yuki Shiino

Kentaro Hara

unread,
Nov 16, 2021, 8:48:33 AM11/16/21
to Yuki Shiino, memor...@chromium.org, Takashi Sakamoto, li...@chromium.org, bar...@chromium.org
I don't have any strong opinion but would vote for #2 or #3 :)


--
You received this message because you are subscribed to the Google Groups "memory-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to memory-dev+...@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/memory-dev/CAN0uC_Sy4rwwTs0%2BLnthgWagX%3D8hLLjQwh5Z7S8FnTboK7TN7w%40mail.gmail.com.


--
Kentaro Hara, Tokyo

Bartek Nowierski

unread,
Nov 16, 2021, 8:00:36 PM11/16/21
to Kentaro Hara, Yuki Shiino, memor...@chromium.org, Takashi Sakamoto, li...@chromium.org, Bartek Nowierski
Not a strong opinion either, but I generally prefer longer names to avoid conflicts with something else that happens to abbreviate "pa". #1 is too long though, so I vote for #2.

Benoit Lize

unread,
Nov 19, 2021, 12:38:52 PM11/19/21
to Bartek Nowierski, Kentaro Hara, Yuki Shiino, memor...@chromium.org, Takashi Sakamoto
#2 or #3, but I don't have a strong preference on the bike shed color :)

If we use a long name such as partition_alloc, we may also "at some point" remove some of the "Partition*" prefixes we have in many places (at least when it's not tied to a specific partition, so I would imagine that e.g. PartitionRoot would keep its name).

Bartek Nowierski

unread,
Nov 19, 2021, 5:43:45 PM11/19/21
to Benoit Lize, Kentaro Hara, Yuki Shiino, memor...@chromium.org, Takashi Sakamoto
Another name that crossed my mind was "palloc", but when I googled, another allocator under this name exists... dunno if that matters.

Yuki Shiino

unread,
Nov 24, 2021, 2:16:38 AM11/24/21
to Bartek Nowierski, Benoit Lize, Kentaro Hara, Yuki Shiino, memor...@chromium.org, Takashi Sakamoto
Thanks for voting.  It looks that everyone is happy with #2 "partition_alloc" as the namespace name of the new PartitionAlloc library, and no other recommendation so far.  Then, let us go with "partition_alloc".

Cheers,
Yuki Shiino


2021年11月20日(土) 7:43 Bartek Nowierski <bar...@chromium.org>:
Reply all
Reply to author
Forward
0 new messages