labath added inline comments.
================
Comment at:
unittests/Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp:400
+
+ HandlePacket(server, "jTraceStart:{\"buffersize\" : 8192,\"jparams\" :
{\"psb\" : 1,\"tracetech\" : \"intel-pt\"},\"metabuffersize\" :
8192,\"threadid\" : 35,\"type\" : 1}", "1");
+ ASSERT_EQ(result.get(),1);
----------------
ravitheja wrote:
> labath wrote:
> > ravitheja wrote:
> > > clayborg wrote:
> > > > Use the R"( so you don't have to desensitize the double quotes and so
> > > > you can format nicely:
> > > >
> > > > ```
> > > > const char *json = R"(
> > > > jTraceStart: {
> > > > "buffersize" : 8192,
> > > > "metabuffersize" : 8192,
> > > > "threadid" : 35,
> > > > "type" : 1,
> > > > "jparams" : {
> > > > "psb" : 1,
> > > > "tracetech" : "intel-pt"
> > > > }
> > > > })";
> > > > HandlePacket(server, json, "1");
> > > > ```
> > > That won't match coz the new lines and extra spaces cause mismatch in the
> > > packet string produced. Also unfortunately the Dump Function in the
> > > StrucuturedData::Dictionary adds spaces around the colons between "Key" :
> > > Value , so removing all whitespaces won't work. Now I can still add some
> > > functions to format the expexted string according to the one printed by
> > > Dump function, but that would be more code.
> > I've been wanting to add make these tests smarter now that they are getting
> > more use. I'll take a todo item here to add a smarter way to compare json
> > fields here. For now you can at least do the R" thing Greg suggests.
> Yes I can do the R thing but then i can't format it nicely, as Greg suggested.
Yes, but it will still avoid the need for escaping quotes, which is an
improvement in itself.
https://reviews.llvm.org/D32585
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
