This is the mail archive of the systemtap@sourceware.org mailing list for the systemtap project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug translator/13999] pretty-printing shouldn't use %c


http://sourceware.org/bugzilla/show_bug.cgi?id=13999

Josh Stone <jistone at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |jistone at redhat dot com
         AssignedTo|systemtap at sourceware dot |jistone at redhat dot com
                   |org                         |

--- Comment #1 from Josh Stone <jistone at redhat dot com> 2012-04-19 18:36:56 UTC ---
(In reply to comment #0)
> We either need a printf formatting variant that escapes non-isprint()
> chars (%C?), or the pretty-printer needs to switch to a non-character output
> such as %u.

How about using the special flag for this, i.e. "%#c"?  This has the advantage
that it's already parsed throughout, though it currently only has meaning for
numbers.  The actual escapes can be identical to _stp_text_str(), except for
allowing '"' as-is and quoting '\'' instead.  (Or just escape both quotes in
either case, and share the code.)

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]