-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add formatted output generation on top of BufferWriter. #18
Conversation
lib/ts/bwprint.cc
Outdated
prefix = "0B"_sv; | ||
n = detail::bwf_to_radix<2>(i, buff, sizeof(buff), detail::UPPER_DIGITS); | ||
break; | ||
break; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra break?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dang, I thought I'd fixed that.
lib/ts/bwprint.cc
Outdated
case 'x': | ||
if (spec._radix_lead_p) | ||
prefix = "0x"_sv; | ||
n = detail::bwf_to_radix<16>(i, buff, sizeof(buff), detail::LOWER_DIGITS); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Putting braces around the body of the if
will fix this weird formatting and also match our style guide.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haha, yeah, that's some serious ugly there, isn't it?
lib/ts/bwprint.cc
Outdated
if (!sz) | ||
return; | ||
} else { | ||
throw std::invalid_argument("Precision mark without precision"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generic use of exceptions is questionable in a source base that has not be upgraded to exception safety.
lib/ts/bwprint.cc
Outdated
|
||
ts::BWFormat::~BWFormat() | ||
{ | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why? Just, why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Habit.
I really don't like the duplication between |
Now the log-headers request will propagate upstream even when logging is not enabled on the local box. This makes it easier to use on higher tier caches. Also added documentation.
The same element is fetched in test one and 5. On a loaded system it could be in cache for a second and report age:1 instead of age:0
This is a bug introduced with PR apache#2335. The `if` is to avoid unnecessary works when buffering the post body, but since redirects also need to execute the logic flow and the request info may be changed during the process of redirects. We need to extract the request info every time redirects happen.
b2f3929
to
86b6cee
Compare
Change 'const std::string &' parameters to virtual member functions of the TranformationPlugin class to ts::string_view. This allows derived classes of TranformationPlugin to avoid using std::string in some cases, resulting in less copying and dynamic memory allocation.
ad9d2f5
to
82fc0f9
Compare
Do not require lhiredis on all binaries when building ssl_session_reuse
No description provided.