On 4/17/2018 9:04 PM, Junio C Hamano wrote:
Stefan Beller <sbeller@xxxxxxxxxx> writes:
What's the doneness of this thing? I didn't recall seeing any
response, especially ones that demonstrated the reviewer carefully
read and thought about the issues surrounding the code. Not that I
spotted any problems in these patches myself, though.
Stolee and Brandon provided a "quick LGTM" type of review
https://public-inbox.org/git/20180409232536.GB102627@xxxxxxxxxx/
https://public-inbox.org/git/9ddfee7e-025a-79c9-8d6b-700c65a14067@xxxxxxxxx/
Yup. Giving positive reviews is harder than giving constructive
criticism. Much harder.
As readers cannot tell from a "quick LGTM" between "I didn't read it
but it did not smell foul" and "I read it thoroughly, understood how
the solution works, it was presented well, and agree with the design
and implementation---there is nothing to add", the reviewers need to
come up with some way to express that it is the latter case rather
than the former.
I would not claim that I've perfected my technique to do so, but
when responding to such a "good" series, I rephrase the main idea in
the series in my own words to show that I as a reviewer read the
series well enough to be able to do so, perhaps with comparison with
possible alternatives I could think of and dicussion to argue that
the solution presented in the series is better, in an attempt to
demonstrate that I am qualified to say "this one is good" with good
enough understanding of both the issue the series addresses and the
solution in the series.
I'm sorry that my second message was terse. My response to v1 [1] was
> I looked through these patches and only found one set of whitespace >
errors. Compiles and tests fine on my machine. > > Reviewed-by: Derrick
Stolee <dstolee@xxxxxxxxxxxxx> So, I pulled the code, went through it
patch-by-patch, and saw that the transformations were made using the
established pattern. The second review was to chime in that my v1
comments had been addressed. Thanks, -Stolee
[1]
https://public-inbox.org/git/6c319100-df47-3b8d-8661-24e4643ada09@xxxxxxxxx/
