[erlang-patches] edoc_lib: padding escaped characters with a leading 0, when their value is one-digit long

Richard Carlsson carlsson.richard@REDACTED
Thu Aug 18 14:51:37 CEST 2011


On 08/14/2011 05:43 PM, Alfonso De Gregorio wrote:
> When escaping characters less than or equal to 9, the escape_byte/1
> does not pad the result with a leading $0 (e.g., "\x01" is escaped in
> %1 and not in %01). The latter would be the expected representation of
> binary data in a URI according to RFC 3986 (and the specs it updates),
> where the percent encoding mechanism is standardized.
>
> The fix is tiny. Being in doubt about the preferred style to follow
> for OTP, here are two proposals:
>
> [...]
>
> My preference goes to the former.

Thanks for the report, but I preferred neither suggestion. If the code 
should output two characters always, then it is better to rewrite it 
completely. (Also, in your first suggestion, a hex number is 1 char if 
it is =< 15, not =< 9.)

I have included the change in the edoc development repository and will 
push it to OTP later:

https://github.com/richcarl/edoc/commit/1a52f6fcbfe37aaa9a5f72eebfe73c8087f0640e

    /Richard



More information about the erlang-patches mailing list