Code review comment for lp://staging/~pete-woods/net-cpp/base64

Revision history for this message
Thomas Voß (thomas-voss) wrote :

> Indeed, the symbols file is missing most of the stuff I'd expect it to
> contain, e.g. url_escape doesn't appear in it. Maybe it has the wrong name,
> and is being ignored?

Might well be the case, and we can fix that in a follow-up MP. With that: LGTM.

review: Approve

« Back to merge proposal