On 2021-01-26 1:07 p.m., Laszlo Ersek wrote:
On 01/26/21 20:15, Ankur Arora wrote:
On 2021-01-26 11:01 a.m., Laszlo Ersek wrote:
I'll continue the review later this week.
Acking the comments above.
Thank you!
Meanwhile let me reprocess the series in light of the comments above.
I'm at such a point now, during the v5 review, that I think I can easily
re-sync.
In general, I don't mind the posting of a new version of a series
mid-review, *IF* we agree about it in advance.
If you prefer to post a v6, for addressing the comments I've made thus
far, I'm OK with that. If you'd like me to continue reviewing v5, I'm
also OK with that.
So it's up to you -- please state your decision, so that I know if I
should proceed with v5 (later this week), or wait for v6.
I think I would prefer to send v6. Looking at the v5 comments so far, I'm
sure that there's a lot of non conforming coding style issues.
Addressing them now (or at least a hopefully significant subset) would
probably save time.
I'm looking at sending these out by Thursday morning PT, and given that
you plan to continue later this week, sounds like it might not lose too
much review time either.
Thanks
Ankur
Thanks!
Laszlo
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#70812): https://edk2.groups.io/g/devel/message/70812
Mute This Topic: https://groups.io/mt/80125308/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
