<div dir="auto"><div><br><div data-smartmail="gmail_signature"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">пн, 17 Ñ„евр. 2020 Ð³., 17:06 Andrey Savchenko &lt;<a href="mailto:bircoph@altlinux.org">bircoph@altlinux.org</a>&gt;:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, 17 Feb 2020 11:41:36 +0300 (MSK) Ivan Zakharyaschev wrote:<br>
&gt; ÐŸÐ¾ÑÐ²Ð¸Ð»Ð°ÑÑŒ Ð¸Ð´ÐµÑ, Ñ‡Ñ‚о ÐµÑÑ‚ÑŒ ÑÐ¿Ð¾ÑÐ¾Ð± Ð·Ð´ÐµÑÑŒ Ð½Ðµ Ð¿Ñ€Ð¾ÑÑ‚о Ð¾Ñ‚ветственность Ð²Ð¾Ð·Ð»Ð¾Ð¶Ð¸Ñ‚ÑŒ <br>
&gt; Ð½Ð° Ð¿Ñ€Ð¾Ð³Ñ€Ð°Ð¼Ð¼Ð¸ÑÑ‚а Ð½Ð° ÑÐ»Ð¾Ð²Ð°Ñ… (вызывать Allocate() Ñ Ð½ÐµÐ½ÑƒÐ»ÐµÐ²Ñ‹Ð¼ Ð°Ñ€Ð³ÑƒÐ¼ÐµÐ½Ñ‚ом), Ð½Ð¾ <br>
&gt; Ð¸ Ñ„ормально Ð¿Ñ€ÐµÐ´Ð»Ð¾Ð¶Ð¸Ñ‚ÑŒ Ñ‚акой ÑÐ¿Ð¾ÑÐ¾Ð± Ð¿Ñ€Ð¾Ð³Ñ€Ð°Ð¼Ð¼Ð¸Ñ€Ð¾Ð²Ð°Ð½Ð¸Ñ, Ð³Ð´Ðµ Ð½Ðµ Ð±ÑƒÐ´ÐµÑ‚ <br>
&gt; Ð²Ð¾Ð·Ð¼Ð¾Ð¶Ð½Ð¾ÑÑ‚и Ð´ÐµÐ»Ð°Ñ‚ÑŒ Ð½Ðµ Ñ‚ак, ÐºÐ°Ðº Ð·Ð°Ð´ÑƒÐ¼Ð°Ð½Ð¾.<br>
&gt; <br>
&gt; Ð”а Ð¸ Ð¿Ð¾Ð¿Ñ€Ð¾Ñ‰Ðµ Ð²Ñ‹Ð·Ð¾Ð²Ñ‹ Ð¿Ð¾Ð»ÑƒÑ‡Ð°ÑŽÑ‚ся.<br>
&gt; <br>
&gt; Ð’етка alloc-templates Ð² <br>
&gt; <a href="http://git.altlinux.org/people/imz/packages/apt.git" rel="noreferrer noreferrer" target="_blank">http://git.altlinux.org/people/imz/packages/apt.git</a><br>
&gt; <br>
&gt; (Развивая ÑÑ‚о, Ð¼Ð¾Ð¶Ð½Ð¾ RawAllocate() Ð¿Ñ€ÐµÐ²Ñ€Ð°Ñ‚ить Ð² ÐºÐ°ÐºÐ¾Ð¹-нибудь<br>
&gt; template&lt;typename T&gt; RawAllocateArray&lt;T&gt;(unsigned long ItemCount),<br>
&gt; Ñ‚ам, Ð¿Ñ€Ð°Ð²Ð´Ð°, Pools Ð½ÐµÑ‚ривиально Ð½Ð°Ð´Ð¾ Ð±ÑƒÐ´ÐµÑ‚ Ð¿ÐµÑ€ÐµÐ´ÐµÐ»Ð°Ñ‚ÑŒ.<br>
&gt; <br>
&gt; ÐŸÑ€Ð¸ÑˆÐ»Ð¾ÑÑŒ, Ð¿Ñ€Ð°Ð²Ð´Ð°, Ð¿ÐµÑ€ÐµÐ½ÐµÑÑ‚и Ð¾Ð¿Ñ€ÐµÐ´ÐµÐ»ÐµÐ½Ð¸Ðµ Ð¸Ð· .cc Ð² .h Ð¸ Ð¿Ð°Ñ‚чи Ð´Ð°Ð»ÑŒÐ½ÐµÐ¹ÑˆÐ¸Ðµ <br>
&gt; Ð½ÐµÑƒÐ´Ð¾Ð±Ð½Ð¾ Ð¿Ñ€Ð¸ÐºÐ»Ð°Ð´Ñ‹Ð²Ð°Ñ‚ÑŒ; Ð½Ð¾ Ð¼Ð¾Ð¶Ð½Ð¾ Ð¸Ñ… ifdef-ами Ð¾Ð±Ð»Ð¾Ð¶Ð¸Ñ‚ÑŒ Ð² .cc Ð¸ Ð·Ð°Ð¸Ð½ÐºÐ»ÑƒÐ´Ð¸Ñ‚ÑŒ <br>
&gt; Ð² .h.)<br>
&gt; <br>
&gt; From 89499e5810575336e6a3b02d71d6c323321a0eca Mon Sep 17 00:00:00 2001<br>
&gt; From: Ivan Zakharyaschev &lt;<a href="mailto:imz@altlinux.org" target="_blank" rel="noreferrer">imz@altlinux.org</a>&gt;<br>
&gt; Date: Fri, 14 Feb 2020 04:04:42 +0300<br>
&gt; Subject: [PATCH] don&#39;t pass compile-time constants as an argument to<br>
&gt;  DynamicMMap::Allocate()<br>
&gt; <br>
&gt; The argument was used for the size of the item we want to allocate.<br>
&gt; <br>
&gt; Instead of an argument, use a parameter for the template DynamicMMap::Allocate()<br>
&gt; which denotes the type of the item.<br>
&gt; <br>
&gt; Now, the truth of our assertion that the size is not zero is evident<br>
&gt; (and checked by the compiler).<br>
&gt; <br>
&gt; Note that this change may also potentially lead to more optimized code<br>
&gt; due to compile-time specialization for each particular size.<br>
&gt; ---<br>
&gt;  apt/apt-pkg/contrib/mmap.h | 12 ++++++------<br>
&gt;  apt/apt-pkg/pkgcachegen.cc | 19 ++++++++++---------<br>
&gt;  apt/apt-pkg/pkgcachegen.h  |  3 ++-<br>
&gt;  3 files changed, 18 insertions(+), 16 deletions(-)<br>
&gt; <br>
&gt; diff --git a/apt/apt-pkg/contrib/mmap.h b/apt/apt-pkg/contrib/mmap.h<br>
&gt; index 5b14ac9d9..7ec0fb8b1 100644<br>
&gt; --- a/apt/apt-pkg/contrib/mmap.h<br>
&gt; +++ b/apt/apt-pkg/contrib/mmap.h<br>
&gt; @@ -94,7 +94,8 @@ class DynamicMMap : public MMap<br>
&gt;  <br>
&gt;  Â  Â // Allocation<br>
&gt;  Â  Â std::experimental::optional&lt;unsigned long&gt; RawAllocate(unsigned long Size,unsigned long Aln = 0);<br>
&gt; -  Â std::experimental::optional&lt;unsigned long&gt; Allocate(unsigned long ItemSize);<br>
&gt; +  Â template&lt;typename T&gt;<br>
&gt; +  Â std::experimental::optional&lt;unsigned long&gt; Allocate();<br>
&gt;  Â  Â std::experimental::optional&lt;unsigned long&gt; WriteString(const char *String,unsigned long Len = std::numeric_limits&lt;unsigned long&gt;::max());<br>
&gt;  Â  Â inline std::experimental::optional&lt;unsigned long&gt; WriteString(const string &amp;S) {return WriteString(S.c_str(),S.length());};<br>
&gt;  Â  Â void UsePools(Pool &amp;P,unsigned int Count) {Pools = &amp;P; PoolCount = Count;};<br>
&gt; @@ -107,17 +108,16 @@ class DynamicMMap : public MMap<br>
&gt;  /* Templates */<br>
&gt;  <br>
&gt;  #include &lt;apt-pkg/error.h&gt;<br>
&gt; -#include &lt;cassert&gt;<br>
&gt;  <br>
&gt;  // DynamicMMap::Allocate - Pooled aligned allocation  Â  Â  Â  Â  Â  Â  Â  Â /*{{{*/<br>
&gt;  // ---------------------------------------------------------------------<br>
&gt;  /* This allocates an Item of size ItemSize so that it is aligned to its<br>
&gt;  Â  Â size in the file. */<br>
&gt; -std::experimental::optional&lt;unsigned long&gt; DynamicMMap::Allocate(unsigned long ItemSize)<br>
&gt; +template&lt;typename T&gt;<br>
&gt; +std::experimental::optional&lt;unsigned long&gt; DynamicMMap::Allocate()<br>
&gt;  {<br>
&gt; -  Â assert(ItemSize != 0); /* Actually, we are always called with sizeof(...)<br>
&gt; -  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â compile-time non-zero constant as the argument.<br>
&gt; -  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  */<br>
&gt; +  Â constexpr unsigned long ItemSize = sizeof(T);<br>
&gt; +  Â static_assert(ItemSize != 0);<br>
<br>
Давайте Ð½Ðµ Ð·Ð°ÐºÐ»Ð°Ð´Ñ‹Ð²Ð°Ñ‚ься Ð½Ð° C++17:<br>
<a href="https://en.cppreference.com/w/cpp/language/static_assert" rel="noreferrer noreferrer" target="_blank">https://en.cppreference.com/w/cpp/language/static_assert</a></blockquote></div></div><div dir="auto"><br></div><div dir="auto">+1.</div><div dir="auto"><br></div><div dir="auto">Чтобы Ð½Ðµ Ð±Ñ‹Ð»Ð¾ Ð¿Ñ€Ð¾Ð±Ð»ÐµÐ¼ Ñ Ð±ÐµÐºÐ¿Ð¾Ñ€Ñ‚ированием. Ð’ Ñ‚ом Ñ‡Ð¸ÑÐ»Ðµ Ð½Ð° e2k. </div><div dir="auto"><br></div><div dir="auto">Rgrds, ÐÐ»ÐµÐºÑÐµÐ¹</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
static_assert ( bool_constexpr , message )  Â  Â  (since C++11)<br>
static_assert ( bool_constexpr )  Â  Â  Â  Â  Â  Â  Â  (since C++17)<br>
<br>
Так Ñ‡Ñ‚о Ð¿Ñ€Ð¾ÑˆÑƒ Ð¸ÑÐ¿Ð¾Ð»ÑŒÐ·Ð¾Ð²Ð°Ñ‚ÑŒ Ð¿ÐµÑ€Ð²ÑƒÑŽ Ñ„Ð¾Ñ€Ð¼Ñƒ (C++11 Ñ ÑÐ¾Ð¾Ð±Ñ‰ÐµÐ½Ð¸ÐµÐ¼)<br>
и ÑƒÐ±ÐµÐ´Ð¸Ñ‚ся, Ñ‡Ñ‚о ÑÐ²Ð½Ð¾ Ð·Ð°Ð´Ð°Ð½ ÐºÐ°Ðº Ð¼Ð¸Ð½Ð¸Ð¼ÑƒÐ¼ -std=c++11/gnu++11.<br>
<br>
&gt;  <br>
&gt;  Â  Â // Look for a matching pool entry<br>
&gt;  Â  Â Pool *I;<br>
&gt; diff --git a/apt/apt-pkg/pkgcachegen.cc b/apt/apt-pkg/pkgcachegen.cc<br>
&gt; index 4bbab07bf..2a75f31e6 100644<br>
&gt; --- a/apt/apt-pkg/pkgcachegen.cc<br>
&gt; +++ b/apt/apt-pkg/pkgcachegen.cc<br>
&gt; @@ -112,8 +112,9 @@ std::experimental::optional&lt;unsigned long&gt; pkgCacheGenerator::WriteStringInMap(c<br>
&gt;  Â  Â return Map.WriteString(String);<br>
&gt;  }<br>
&gt;  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â /*}}}*/<br>
&gt; -std::experimental::optional&lt;unsigned long&gt; pkgCacheGenerator::AllocateInMap(unsigned long size) {/*{{{*/<br>
&gt; -  Â return Map.Allocate(size);<br>
&gt; +template&lt;typename T&gt;<br>
&gt; +std::experimental::optional&lt;unsigned long&gt; pkgCacheGenerator::AllocateInMap() {/*{{{*/<br>
&gt; +  Â return Map.Allocate&lt;T&gt;();<br>
&gt;  }<br>
&gt;  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â /*}}}*/<br>
&gt;  // CacheGenerator::MergeList - Merge the package list  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  /*{{{*/<br>
&gt; @@ -376,7 +377,7 @@ bool pkgCacheGenerator::NewPackage(pkgCache::PkgIterator &amp;Pkg, const string &amp;Nam<br>
&gt;  #endif<br>
&gt;  Â  Â  Â  Â <br>
&gt;  Â  Â // Get a structure<br>
&gt; -  Â const auto Package = AllocateInMap(sizeof(pkgCache::Package));<br>
&gt; +  Â const auto Package = AllocateInMap&lt;pkgCache::Package&gt;();<br>
&gt;  Â  Â const auto idxName = WriteStringInMap(Name);<br>
&gt;  Â  Â if ((!Package) || (!idxName))<br>
&gt;  Â  Â  Â  return false;<br>
&gt; @@ -429,7 +430,7 @@ bool pkgCacheGenerator::NewFileVer(pkgCache::VerIterator &amp;Ver,<br>
&gt;  Â  Â  Â  return true;<br>
&gt;  Â  Â <br>
&gt;  Â  Â // Get a structure<br>
&gt; -  Â const auto VerFile = AllocateInMap(sizeof(pkgCache::VerFile));<br>
&gt; +  Â const auto VerFile = AllocateInMap&lt;pkgCache::VerFile&gt;();<br>
&gt;  Â  Â if (!VerFile)<br>
&gt;  Â  Â  Â  return false;<br>
&gt;  Â  Â <br>
&gt; @@ -460,7 +461,7 @@ std::experimental::optional&lt;unsigned long&gt; pkgCacheGenerator::NewVersion(pkgCach<br>
&gt;  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â unsigned long Next)<br>
&gt;  {<br>
&gt;  Â  Â // Get a structure<br>
&gt; -  Â const auto Version = AllocateInMap(sizeof(pkgCache::Version));<br>
&gt; +  Â const auto Version = AllocateInMap&lt;pkgCache::Version&gt;();<br>
&gt;  Â  Â const auto idxVerStr = WriteStringInMap(VerStr);<br>
&gt;  Â  Â if ((!Version) || (!idxVerStr))<br>
&gt;  Â  Â  Â  return std::experimental::nullopt;<br>
&gt; @@ -487,7 +488,7 @@ bool pkgCacheGenerator::ListParser::NewDepends(pkgCache::VerIterator &amp;Ver,<br>
&gt;  Â  Â pkgCache &amp;Cache = Owner-&gt;Cache;<br>
&gt;  Â  Â <br>
&gt;  Â  Â // Get a structure<br>
&gt; -  Â const auto Dependency = Owner-&gt;AllocateInMap(sizeof(pkgCache::Dependency));<br>
&gt; +  Â const auto Dependency = Owner-&gt;AllocateInMap&lt;pkgCache::Dependency&gt;();<br>
&gt;  Â  Â if (!Dependency)<br>
&gt;  Â  Â  Â  return false;<br>
&gt;  Â  Â <br>
&gt; @@ -574,7 +575,7 @@ bool pkgCacheGenerator::ListParser::NewProvides(pkgCache::VerIterator &amp;Ver,<br>
&gt;  Â  Â  Â  }<br>
&gt;  Â  Â }<br>
&gt;  Â  Â // Get a structure<br>
&gt; -  Â const auto Provides = Owner-&gt;AllocateInMap(sizeof(pkgCache::Provides));<br>
&gt; +  Â const auto Provides = Owner-&gt;AllocateInMap&lt;pkgCache::Provides&gt;();<br>
&gt;  Â  Â if (!Provides)<br>
&gt;  Â  Â  Â  return false;<br>
&gt;  Â  Â /* Note:<br>
&gt; @@ -618,7 +619,7 @@ bool pkgCacheGenerator::SelectFile(const string &amp;File, const string &amp;Site,<br>
&gt;  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  unsigned long Flags)<br>
&gt;  {<br>
&gt;  Â  Â // Get some space for the structure<br>
&gt; -  Â const auto idxFile = AllocateInMap(sizeof(*CurrentFile));<br>
&gt; +  Â const auto idxFile = AllocateInMap&lt;decltype(*CurrentFile)&gt;();<br>
&gt;  Â  Â const auto idxFileName = WriteStringInMap(File);<br>
&gt;  Â  Â const auto idxSite = WriteUniqString(Site);<br>
&gt;  Â  Â const auto idxIndexType = WriteUniqString(Index.GetType()-&gt;Label);<br>
&gt; @@ -676,7 +677,7 @@ std::experimental::optional&lt;unsigned long&gt; pkgCacheGenerator::WriteUniqString(co<br>
&gt;  Â  Â }<br>
&gt;  Â  Â <br>
&gt;  Â  Â // Get a structure<br>
&gt; -  Â const auto Item = AllocateInMap(sizeof(pkgCache::StringItem));<br>
&gt; +  Â const auto Item = AllocateInMap&lt;pkgCache::StringItem&gt;();<br>
&gt;  Â  Â const auto idxString = WriteStringInMap(S, Size);<br>
&gt;  Â  Â if ((!Item) || (!idxString))<br>
&gt;  Â  Â  Â  return std::experimental::nullopt;<br>
&gt; diff --git a/apt/apt-pkg/pkgcachegen.h b/apt/apt-pkg/pkgcachegen.h<br>
&gt; index b789177ef..73252802b 100644<br>
&gt; --- a/apt/apt-pkg/pkgcachegen.h<br>
&gt; +++ b/apt/apt-pkg/pkgcachegen.h<br>
&gt; @@ -39,7 +39,8 @@ class pkgCacheGenerator<br>
&gt;  Â  Â std::experimental::optional&lt;unsigned long&gt; WriteStringInMap(const std::string &amp;String) { return WriteStringInMap(String.c_str(), String.length()); };<br>
&gt;  Â  Â std::experimental::optional&lt;unsigned long&gt; WriteStringInMap(const char *String);<br>
&gt;  Â  Â std::experimental::optional&lt;unsigned long&gt; WriteStringInMap(const char *String, unsigned long Len);<br>
&gt; -  Â std::experimental::optional&lt;unsigned long&gt; AllocateInMap(unsigned long size);<br>
&gt; +  Â template&lt;typename T&gt;<br>
&gt; +  Â std::experimental::optional&lt;unsigned long&gt; AllocateInMap();<br>
&gt;  <br>
&gt;  Â  Â public:<br>
&gt;  Â  Â <br>
<br>
<br>
Best regards,<br>
Andrew Savchenko<br>
_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.altlinux.org" target="_blank" rel="noreferrer">Devel@lists.altlinux.org</a><br>
<a href="https://lists.altlinux.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">https://lists.altlinux.org/mailman/listinfo/devel</a></blockquote></div></div></div>