[Lldb-commits] [PATCH] D53749: [PDB] Fix `SymbolFilePDBTests` after r345313

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Fri Oct 26 02:30:09 PDT 2018


For trivial changes it's ok to submit without review.  This is true for
cleanup and trivial refactor, but especially for build break like this
one., and even more so if you consider yourself code owner in the
corresponding code area.

On Fri, Oct 26, 2018 at 2:29 AM Aleksandr Urakov via Phabricator <
reviews at reviews.llvm.org> wrote:

> aleksandr.urakov added a comment.
>
> Ok, thanks! I didn't know about such rule. I'll reduce usages of auto :)
>
> As for such an obvious changes, is it ok to commit them without a review?
> Or is it still preferable to create reviews for them?
>
>
> Repository:
>   rLLDB LLDB
>
> https://reviews.llvm.org/D53749
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20181026/e84abbca/attachment.html>


More information about the lldb-commits mailing list