Apply suggestions from code review

Co-authored-by: Louis Tricot <[email protected]>

Author: dmehala

src/datadog/platform_util.cpp

@@ -314,6 +314,7 @@ Optional<ino_t> get_inode(std::string_view path) {
 // whether the binary is running in host or not. However, it does not work when
 // running in a Docker in Docker environment.
 bool is_running_in_host_namespace() {
+  // linux procfs file that represents the cgroup namespace of the current process
   if (auto inode = get_inode("/proc/self/ns/cgroup")) {
     return *inode == HOST_CGROUP_NAMESPACE_INODE;
   }
@@ -339,7 +340,7 @@ Optional<Cgroup> get_cgroup_version() {
 Optional<std::string> find_docker_container_id_from_cgroup() {
   constexpr std::string_view cgroup_path = "/proc/self/cgroup";
 
-  auto cgroup_fd = std::ifstream(cgroup_path.data(), std::ios::in);
+  auto cgroup_fd = std::ifstream("/proc/self/cgroup", std::ios::in);
   if (!cgroup_fd.is_open()) return nullopt;
 
   return find_docker_container_id(cgroup_fd);

src/datadog/platform_util.h

@@ -94,7 +94,7 @@ Optional<std::string> find_docker_container_id(std::istream& source);
 
 /// Function to retrieve the container metadata.
 ///
-/// @return A `ContainerMetadata` object containing metadata of the container in
+/// @return A `ContainerID` object containing id of the container in
 /// which the current process is running.
 Optional<ContainerID> get_id();