Hi Sasha, thanks for confirming. Wondering if I should file a github issue
for this kind of trivial fix?

*Rossi*


Sasha Krassovsky <krassovskysa...@gmail.com> 于2023年4月14日周五 01:44写道:

> Hi Rossi,
> That’s a good catch! I _think_ the compiler will automatically emit the
> move because it sees we’re copying from an object that’ll never be used
> again [1], but adding the std::move would be good just to remove any
> ambiguity. Go ahead and make the PR!
>
> Sasha
>
> Move, simply <https://herbsutter.com/2020/02/17/move-simply/>
> herbsutter.com <https://herbsutter.com/2020/02/17/move-simply/>
> [image: 4554b8d24c7f200dc5e2e1b18db1893f.png]
> <https://herbsutter.com/2020/02/17/move-simply/>
> <https://herbsutter.com/2020/02/17/move-simply/>
>
>
> 13 апр. 2023 г., в 10:17, Ruoxi Sun <zanmato1...@gmail.com> написал(а):
>
> Hi folks, when reading the swiss join code, I just noticed a small piece
> probably missing a `std::move()` call.
>
> See here:
>
> https://github.com/zanmato1984/arrow/commit/10f43c357db7a0287c642a23e78027cb9cde6f25
>
> If so, I think I can proceed to PR it.
>
> Thanks.
>
> *Rossi Sun*
>
>

Reply via email to