|
From: | lemzwerg |
Subject: | Re: Issue XXXX: Clean up to_string () etc. (issue 583320043 by address@hidden) |
Date: | Sat, 11 Jan 2020 14:20:34 -0800 |
LGTM, thanks! https://codereview.appspot.com/583320043/diff/581420044/flower/file-name.cc File flower/file-name.cc (right): https://codereview.appspot.com/583320043/diff/581420044/flower/file-name.cc#newcode124 flower/file-name.cc:124: s += ext_; any reason to use two lines? In other files you use a single line for similar things. https://codereview.appspot.com/583320043/
[Prev in Thread] | Current Thread | [Next in Thread] |