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

Unified Diff: examples/serialization/main.cc

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: examples/serialization/main.cc
diff --git a/examples/serialization/main.cc b/examples/serialization/main.cc
index 9b0a96d69f698e48159ed952964ac766d65a0a66..381ab9722e7386a173d851c74b9597ec52d1a79f 100644
--- a/examples/serialization/main.cc
+++ b/examples/serialization/main.cc
@@ -19,8 +19,7 @@ int main() {
char buf[1000];
MOJO_CHECK(in.Serialize(buf, sizeof(buf)));
-
- out.Deserialize(buf);
+ MOJO_CHECK(out.Deserialize(buf, sizeof(buf)));
MOJO_CHECK(out.a == 1);
MOJO_CHECK(out.b == 2.0f);
MOJO_CHECK(out.c == "hello world!");
« no previous file with comments | « no previous file | mojo/public/cpp/bindings/lib/bindings_internal.h » ('j') | mojo/public/cpp/bindings/lib/bindings_internal.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698