PPAPI: Remove out-dated or unnecessary TODOs from public files.

ppapi/cpp/var.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ppapi/cpp/var.h"
 
 #include <stdio.h>
 #include <string.h>
 
 #include <algorithm>
@@ skipping 201 matching lines @@
 
 std::string Var::DebugString() const {
   char buf[256];
   if (is_undefined()) {
     snprintf(buf, sizeof(buf), "Var(UNDEFINED)");
   } else if (is_null()) {
     snprintf(buf, sizeof(buf), "Var(NULL)");
   } else if (is_bool()) {
     snprintf(buf, sizeof(buf), AsBool() ? "Var(true)" : "Var(false)");
   } else if (is_int()) {
-    // Note that the following static_cast is necessary because
-    // NativeClient's int32_t is actually "long".
-    // TODO(sehr,polina): remove this after newlib is changed.

[dmichael (off chromium), 2012/03/19 17:41:52]

this static_cast is correct; there's no need to remove it. You should never
assume int32_t is some particular type anyway.

     snprintf(buf, sizeof(buf), "Var(%d)", static_cast<int>(AsInt()));
   } else if (is_double()) {
     snprintf(buf, sizeof(buf), "Var(%f)", AsDouble());
   } else if (is_string()) {
     char format[] = "Var<'%s'>";
     size_t decoration = sizeof(format) - 2;  // The %s is removed.
     size_t available = sizeof(buf) - decoration;
     std::string str = AsString();
     if (str.length() > available) {
       str.resize(available - 3);  // Reserve space for ellipsis.
       str.append("...");
     }
     snprintf(buf, sizeof(buf), format, str.c_str());
   } else if (is_array_buffer()) {
-    // TODO(dmichael): We could make this dump hex. Maybe DebugString should be
-    // virtual?

[dmichael (off chromium), 2012/03/19 17:41:52]

Just a nice-to-have.

     snprintf(buf, sizeof(buf), "Var(ARRAY_BUFFER)");
   } else if (is_object()) {
     snprintf(buf, sizeof(buf), "Var(OBJECT)");
   }
   return buf;
 }
 
 }  // namespace pp