Commit 5e1b7327 authored by Ameya Limaye's avatar Ameya Limaye Committed by Facebook Github Bot

Make sequence number in AtomicBatchDispatcher<....>::Token non-const

Summary:
Make sequence number in AtomicBatchDispatcher<....>::Token non-const:
- The field is already private and having it as const prevents a
  move assignment operator from getting defined

Reviewed By: yfeldblum

Differential Revision: D4114851

fbshipit-source-id: 9f7a4003daff02c007c01747cca6ae9da17fcb79
parent b9ac5511
...@@ -146,7 +146,7 @@ template <typename InputT, typename ResultT> ...@@ -146,7 +146,7 @@ template <typename InputT, typename ResultT>
AtomicBatchDispatcher<InputT, ResultT>::Token::Token( AtomicBatchDispatcher<InputT, ResultT>::Token::Token(
std::shared_ptr<DispatchBaton> baton, std::shared_ptr<DispatchBaton> baton,
size_t sequenceNumber) size_t sequenceNumber)
: baton_(std::move(baton)), SEQUENCE_NUMBER(sequenceNumber) {} : baton_(std::move(baton)), sequenceNumber_(sequenceNumber) {}
template <typename InputT, typename ResultT> template <typename InputT, typename ResultT>
Future<ResultT> AtomicBatchDispatcher<InputT, ResultT>::Token::dispatch( Future<ResultT> AtomicBatchDispatcher<InputT, ResultT>::Token::dispatch(
...@@ -156,7 +156,7 @@ Future<ResultT> AtomicBatchDispatcher<InputT, ResultT>::Token::dispatch( ...@@ -156,7 +156,7 @@ Future<ResultT> AtomicBatchDispatcher<InputT, ResultT>::Token::dispatch(
throw std::logic_error( throw std::logic_error(
"Dispatch called more than once on the same Token object"); "Dispatch called more than once on the same Token object");
} }
return baton->getFutureResult(std::move(input), SEQUENCE_NUMBER); return baton->getFutureResult(std::move(input), sequenceNumber_);
} }
template <typename InputT, typename ResultT> template <typename InputT, typename ResultT>
......
...@@ -153,7 +153,7 @@ class AtomicBatchDispatcher { ...@@ -153,7 +153,7 @@ class AtomicBatchDispatcher {
Token& operator=(const Token&) = delete; Token& operator=(const Token&) = delete;
std::shared_ptr<DispatchBaton> baton_; std::shared_ptr<DispatchBaton> baton_;
const size_t SEQUENCE_NUMBER; size_t sequenceNumber_;
}; };
explicit AtomicBatchDispatcher(DispatchFunctionT&& dispatchFunc); explicit AtomicBatchDispatcher(DispatchFunctionT&& dispatchFunc);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment