[libc-commits] [libc] [libc] Add inet_ntop (PR #204143)

[Pavel Labath via libc-commits]

================
@@ -61,5 +68,160 @@ cpp::optional<in_addr_t> inet_addr(const char *cp) {
   return Endian::to_big_endian(result);
 }
 
+namespace {
+
+size_t ipv4_num_bytes(cpp::span<const uint8_t> src) {
+  size_t result = 8; // four digits, three dots and '\0'
+  for (unsigned i = 0; i < 4; ++i)
+    result += (src[i] >= 10) + (src[i] >= 100);
+  return result;
+}
+
+size_t ipv4_to_str_unchecked(cpp::span<const uint8_t> src,
+                             cpp::span<char> dst) {
+  size_t pos = 0;
----------------
labath wrote:

Span doesn't do any bounds checks :/, but even if it did, normally, this function will get inlined into ipv4_to_str, at which point the bound should be a constant visible to the compiler.

https://github.com/llvm/llvm-project/pull/204143