Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(534)

Unified Diff: mojo/public/cpp/bindings/lib/bindings_internal.h

Issue 1800753005: C++ bindings: A struct's Deserialize() now does validation before deserializing. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Address comments. DeserializeWithoutValidation returns void, doesn't take in buf_size. Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/public/cpp/bindings/lib/bindings_internal.h
diff --git a/mojo/public/cpp/bindings/lib/bindings_internal.h b/mojo/public/cpp/bindings/lib/bindings_internal.h
index 7889fa6f9365d01354d50a9f56e453bdfcf9cf76..39d13bc8384606efba931cc491467430880651c3 100644
--- a/mojo/public/cpp/bindings/lib/bindings_internal.h
+++ b/mojo/public/cpp/bindings/lib/bindings_internal.h
@@ -98,6 +98,8 @@ struct IsHandle {
enum { value = std::is_base_of<Handle, H>::value };
};
+// TODO(vardhan): Should RemoveStructPtr<> be internal-only? Should be /the/ way
viettrungluu 2016/03/30 17:49:01 Hmmm. Or is specializing std::pointer_traits the r
vardhan 2016/03/30 22:49:07 yeah, good point; std::pointer_traits will defini
+// to get T from StructPtr<T>?
template <typename T>
struct RemoveStructPtr {
typedef T type;

Powered by Google App Engine
This is Rietveld 408576698