Hi Rossi, 
I think for small PRs like this it is fine to just prefix your PR with “MINOR” 
and not have an associated issue.

Sasha

> On Apr 13, 2023, at 10:48 AM, Ruoxi Sun <zanmato1...@gmail.com> wrote:
> 
> 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 
> <mailto: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
>> herbsutter.com
>> 
>>  <https://herbsutter.com/2020/02/17/move-simply/>Move, simply 
>> <https://herbsutter.com/2020/02/17/move-simply/>
>> herbsutter.com <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 
>>> <mailto: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