Todd Fiala wrote:
If we were to leave the client code the same and switch the qHostInfo
to be hex, as long as we prepended the 0x to the qHostInfo generation
in RNBRemote.cpp and GDBRemoteCommunicationServer.cpp, I think it
would all still work. (Existing older debugserver/lldb-platform would
write decimal in a 0-base client friendly way and get interpreted
correctly, and newer debugserver/lldb-platform/llgs would prepend a 0x
and also get proper client interpretation due to the 0-base client
code. And since the code is the same on the client for old/new, the
matrix of old/new stub and old/new client would probably work. But
I'd never mess with production code just for the benefit of making the
spec look cleaner (and changing the spec in the process). So my vote
would be to leave it as it is.
But that's just my 2 cents ;-)
Cool, cheers. Just wanted to clear it up a little. Need to change my
stub now!
Later
Matt
Member of the CSR plc group of companies. CSR plc registered in England and
Wales, registered number 4187346, registered office Churchill House, Cambridge
Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom
More information can be found at www.csr.com. Keep up to date with CSR on our
technical blog, www.csr.com/blog, CSR people blog, www.csr.com/people, YouTube,
www.youtube.com/user/CSRplc, Facebook,
www.facebook.com/pages/CSR/191038434253534, or follow us on Twitter at
www.twitter.com/CSR_plc.
New for 2014, you can now access the wide range of products powered by aptX at
www.aptx.com.
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits