From aa6dc82f4bdddc9fa9ded310c62eff40c03e101e Mon Sep 17 00:00:00 2001
From: tangwei12 <tangwei12@baidu.com>
Date: Fri, 26 Oct 2018 13:00:44 +0800
Subject: [PATCH] revert changes in protobuf.cc and type_defs

---
 paddle/fluid/framework/framework.proto | 42 +++++++++++++-------------
 paddle/fluid/framework/type_defs.h     |  8 ++---
 2 files changed, 25 insertions(+), 25 deletions(-)

diff --git a/paddle/fluid/framework/framework.proto b/paddle/fluid/framework/framework.proto
index 423fe5e69..2545e6c6f 100644
--- a/paddle/fluid/framework/framework.proto
+++ b/paddle/fluid/framework/framework.proto
@@ -25,17 +25,17 @@ message Version { optional int64 version = 1 [ default = 0 ]; }
 
 enum AttrType {
   INT = 0;
-  LONG = 1;
-  FLOAT = 2;
-  STRING = 3;
-  INTS = 4;
-  LONGS = 5;
-  FLOATS = 6;
-  STRINGS = 7;
-  BOOLEAN = 8;
-  BOOLEANS = 9;
-  BLOCK = 10;
-  BLOCKS = 11;
+  FLOAT = 1;
+  STRING = 2;
+  INTS = 3;
+  FLOATS = 4;
+  STRINGS = 5;
+  BOOLEAN = 6;
+  BOOLEANS = 7;
+  BLOCK = 8;
+  LONG = 9;
+  BLOCKS = 10;
+  LONGS = 11;
 }
 
 // OpDesc describes an instance of a C++ framework::OperatorBase
@@ -46,17 +46,17 @@ message OpDesc {
     required string name = 1;
     required AttrType type = 2;
     optional int32 i = 3;
-    optional int64 l = 4;
-    optional float f = 5;
-    optional string s = 6;
-    repeated int32 ints = 7;
-    repeated int64 longs = 8;
-    repeated float floats = 9;
-    repeated string strings = 10;
-    optional bool b = 11;
-    repeated bool bools = 12;
-    optional int32 block_idx = 13;
+    optional float f = 4;
+    optional string s = 5;
+    repeated int32 ints = 6;
+    repeated float floats = 7;
+    repeated string strings = 8;
+    optional bool b = 10;
+    repeated bool bools = 11;
+    optional int32 block_idx = 12;
+    optional int64 l = 13;
     repeated int32 blocks_idx = 14;
+    optional int64 longs = 15;
   };
 
   message Var {
diff --git a/paddle/fluid/framework/type_defs.h b/paddle/fluid/framework/type_defs.h
index 1cbf6c32a..2de6233a9 100644
--- a/paddle/fluid/framework/type_defs.h
+++ b/paddle/fluid/framework/type_defs.h
@@ -33,10 +33,10 @@ using VariableNameMap = std::map<std::string, std::vector<std::string>>;
 
 // The order should be as same as framework.proto
 using Attribute =
-    boost::variant<boost::blank, int, int64_t, float, std::string,
-                   std::vector<int>, std::vector<int64_t>, std::vector<float>,
-                   std::vector<std::string>, bool, std::vector<bool>,
-                   BlockDesc*, std::vector<BlockDesc*>>;
+    boost::variant<boost::blank, int, float, std::string, std::vector<int>,
+                   std::vector<float>, std::vector<std::string>, bool,
+                   std::vector<bool>, BlockDesc*, int64_t,
+                   std::vector<BlockDesc*>, std::vector<int64_t>>;
 
 using AttributeMap = std::unordered_map<std::string, Attribute>;
 
-- 
GitLab