Re: [Lldb-commits] [PATCH] debugserver: Disable four-char-constants and format-pedantic warnings

2016-10-27 Thread Justin Bogner via lldb-commits
Pavel Labath writes: > On 19 October 2016 at 00:20, Justin Bogner wrote: >> Pavel Labath writes: >>> Thanks for the patch. Could you submit the patch through phabricator >>> and add Greg Clayton as a reviewer. >> >> Maybe later. I don't have time to fight with phabrica

Re: [Lldb-commits] [PATCH] debugserver: Disable four-char-constants and format-pedantic warnings

2016-10-19 Thread Pavel Labath via lldb-commits
On 19 October 2016 at 00:20, Justin Bogner wrote: > Pavel Labath writes: >> Thanks for the patch. Could you submit the patch through phabricator >> and add Greg Clayton as a reviewer. > > Maybe later. I don't have time to fight with phabricator today. > >> That said, my

Re: [Lldb-commits] [PATCH] debugserver: Disable four-char-constants and format-pedantic warnings

2016-10-18 Thread Justin Bogner via lldb-commits
Pavel Labath writes: > Thanks for the patch. Could you submit the patch through phabricator > and add Greg Clayton as a reviewer. Maybe later. I don't have time to fight with phabricator today. > That said, my preference would be to actually fix those warnings > instea

Re: [Lldb-commits] [PATCH] debugserver: Disable four-char-constants and format-pedantic warnings

2016-10-18 Thread Pavel Labath via lldb-commits
Hi, Thanks for the patch. Could you submit the patch through phabricator and add Greg Clayton as a reviewer. That said, my preference would be to actually fix those warnings instead of silencing them. pl On 17 October 2016 at 08:24, Justin Bogner via lldb-commits wrot