Committed as https://github.com/DynamoRIO/dynamorio/commit/f68d392252f001bcbf8b632600326d7430677321 Final commit log: --------------- Remove two non-UTF8 characters in retcheck.c Review-URL: https://codereview.appspot.com/305690043 ...
7 years, 7 months ago
(2016-09-30 16:18:17 UTC)
#2
On 2016/09/30 16:18:17, zhaoqin wrote: > Remove two non-UTF8 characters in retcheck.c The code change ...
7 years, 7 months ago
(2016-09-30 18:17:16 UTC)
#3
On 2016/09/30 16:18:17, zhaoqin wrote:
> Remove two non-UTF8 characters in retcheck.c
The code change is good but the comment seems inaccurate: they are ascii and
utf8 as they are single chars of code 0x18. They are just non-printable.
On 2016/09/30 18:17:16, bruening wrote: > On 2016/09/30 16:18:17, zhaoqin wrote: > > Remove two ...
7 years, 7 months ago
(2016-09-30 19:00:20 UTC)
#4
On 2016/09/30 18:17:16, bruening wrote:
> On 2016/09/30 16:18:17, zhaoqin wrote:
> > Remove two non-UTF8 characters in retcheck.c
>
> The code change is good but the comment seems inaccurate: they are ascii and
> utf8 as they are single chars of code 0x18. They are just non-printable.
sorry, I was using the error message from a build system which says non-UTF8
characters.
Issue 305690043: TBR: Remove two non-UTF8 characters in retcheck.c
Created 7 years, 7 months ago by zhaoqin
Modified 7 years, 7 months ago
Reviewers: bruening
Base URL:
Comments: 0