Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: src/shared/Communication.h

Issue 11013110: Cleanup (Closed)
Left Patch Set: More refactoring. Removing main thread, tab counting. Implementing SetPref and GetPref. Addressing … Created July 9, 2013, 12:59 p.m.
Right Patch Set: More beautification and addressing comments Created July 29, 2013, 12:13 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « src/plugin/Resource.h ('k') | src/shared/Communication.cpp » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 #ifndef COMMUNICATION_H 1 #ifndef COMMUNICATION_H
2 #define COMMUNICATION_H 2 #define COMMUNICATION_H
3 3
4 #include <memory> 4 #include <memory>
5 #include <sstream> 5 #include <sstream>
6 #include <stdexcept> 6 #include <stdexcept>
7 #include <stdint.h> 7 #include <stdint.h>
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 #include <Windows.h> 10 #include <Windows.h>
(...skipping 17 matching lines...) Expand all
28 PROC_GET_PREF 28 PROC_GET_PREF
29 }; 29 };
30 enum ValueType : uint32_t { 30 enum ValueType : uint32_t {
31 TYPE_PROC, TYPE_STRING, TYPE_WSTRING, TYPE_INT64, TYPE_INT32, TYPE_BOOL 31 TYPE_PROC, TYPE_STRING, TYPE_WSTRING, TYPE_INT64, TYPE_INT32, TYPE_BOOL
32 }; 32 };
33 typedef uint32_t SizeType; 33 typedef uint32_t SizeType;
34 34
35 class InputBuffer 35 class InputBuffer
36 { 36 {
37 public: 37 public:
38 InputBuffer() : buffer(), hasType(false) {}
38 InputBuffer(const std::string& data) : buffer(data), hasType(false) {} 39 InputBuffer(const std::string& data) : buffer(data), hasType(false) {}
40 InputBuffer(const InputBuffer& copy)
41 {
42 hasType = copy.hasType;
43 buffer = std::istringstream(copy.buffer.str());
44 currentType = copy.currentType;
45 }
39 InputBuffer& operator>>(ProcType& value) { return Read(value, TYPE_PROC); } 46 InputBuffer& operator>>(ProcType& value) { return Read(value, TYPE_PROC); }
40 InputBuffer& operator>>(std::string& value) { return ReadString(value, TYPE_ STRING); } 47 InputBuffer& operator>>(std::string& value) { return ReadString(value, TYPE_ STRING); }
41 InputBuffer& operator>>(std::wstring& value) { return ReadString(value, TYPE _WSTRING); } 48 InputBuffer& operator>>(std::wstring& value) { return ReadString(value, TYPE _WSTRING); }
42 InputBuffer& operator>>(int64_t& value) { return Read(value, TYPE_INT64); } 49 InputBuffer& operator>>(int64_t& value) { return Read(value, TYPE_INT64); }
43 InputBuffer& operator>>(int32_t& value) { return Read(value, TYPE_INT32); } 50 InputBuffer& operator>>(int32_t& value) { return Read(value, TYPE_INT32); }
44 InputBuffer& operator>>(bool& value) { return Read(value, TYPE_BOOL); } 51 InputBuffer& operator>>(bool& value) { return Read(value, TYPE_BOOL); }
52 InputBuffer& operator=(const InputBuffer& copy)
53 {
54 hasType = copy.hasType;
55 buffer = std::istringstream(copy.buffer.str());
56 currentType = copy.currentType;
57 return *this;
58 }
59 ValueType GetType();
45 private: 60 private:
46 std::istringstream buffer; 61 std::istringstream buffer;
47 ValueType currentType; 62 ValueType currentType;
48 bool hasType; 63 bool hasType;
49 64
50 void CheckType(ValueType expectedType); 65 void CheckType(ValueType expectedType);
51 66
52 template<class T> 67 template<class T>
53 InputBuffer& ReadString(T& value, ValueType expectedType) 68 InputBuffer& ReadString(T& value, ValueType expectedType)
54 { 69 {
(...skipping 18 matching lines...) Expand all
73 ReadBinary(value); 88 ReadBinary(value);
74 return *this; 89 return *this;
75 } 90 }
76 91
77 template<class T> 92 template<class T>
78 void ReadBinary(T& value) 93 void ReadBinary(T& value)
79 { 94 {
80 buffer.read(reinterpret_cast<char*>(&value), sizeof(T)); 95 buffer.read(reinterpret_cast<char*>(&value), sizeof(T));
81 if (buffer.fail()) 96 if (buffer.fail())
82 throw new std::runtime_error("Unexpected end of input buffer"); 97 throw new std::runtime_error("Unexpected end of input buffer");
98 hasType = false;
83 } 99 }
84 }; 100 };
85 101
86 class OutputBuffer 102 class OutputBuffer
87 { 103 {
88 public: 104 public:
89 OutputBuffer() {} 105 OutputBuffer() {}
90 106
91 // Explicit copy constructor to allow returning OutputBuffer by value 107 // Explicit copy constructor to allow returning OutputBuffer by value
92 OutputBuffer(const OutputBuffer& copy) : buffer(copy.buffer.str()) {} 108 OutputBuffer(const OutputBuffer& copy) : buffer(copy.buffer.str()) {}
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 177
162 InputBuffer ReadMessage(); 178 InputBuffer ReadMessage();
163 void WriteMessage(OutputBuffer& message); 179 void WriteMessage(OutputBuffer& message);
164 180
165 protected: 181 protected:
166 HANDLE pipe; 182 HANDLE pipe;
167 }; 183 };
168 } 184 }
169 185
170 #endif 186 #endif
LEFTRIGHT

Powered by Google App Engine
This is Rietveld