> On Feb. 28, 2017, 11:16 p.m., Greg Mann wrote: > > 3rdparty/stout/tests/base64_tests.cpp > > Lines 48-69 (patched) > > <https://reviews.apache.org/r/56665/diff/3/?file=1642453#file1642453line48> > > > > Could we use strings for these tests that include both of the modified > > alphabet characters, '-' and '_'?
Not easy to find something that's printable, but it's possible with UTF-8. - Jan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56665/#review167145 ----------------------------------------------------------- On March 3, 2017, 3 p.m., Jan Schlicht wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56665/ > ----------------------------------------------------------- > > (Updated March 3, 2017, 3 p.m.) > > > Review request for mesos, Alexander Rojas and Greg Mann. > > > Bugs: MESOS-7001 > https://issues.apache.org/jira/browse/MESOS-7001 > > > Repository: mesos > > > Description > ------- > > Base64 has many variants that use different alphabets for encoding. > "Base 64 Encoding with URL and Filename Safe Alphabet" is a variant > described in RFC 4648. > > > Diffs > ----- > > 3rdparty/stout/include/stout/base64.hpp > 2ac04c4602bc919633a2a480dd2168b7aa301bd7 > 3rdparty/stout/tests/base64_tests.cpp > 32e516861d44c7e3a36e1a29b4d1fe5960684e3b > > > Diff: https://reviews.apache.org/r/56665/diff/4/ > > > Testing > ------- > > make check > > > Thanks, > > Jan Schlicht > >
