[GitHub] [trafficserver] maskit opened a new issue, #10344: CID 1518594: COPY_INSTEAD_OF_MOVE

["maskit (via GitHub)" <gi...@apache.org>]

maskit opened a new issue, #10344:
URL: https://github.com/apache/trafficserver/issues/10344

   inliner-handler.cc
   ```
    31namespace ats
    32{
    33namespace inliner
    34{
    35  Handler::Handler(const TSIOBufferReader r, ats::io::IOSinkPointer &&i)
      	
   CID 1518594 (#1 of 1): COPY_INSTEAD_OF_MOVE (COPY_INSTEAD_OF_MOVE)
   1. copy_constructor_call: i is copied in call to copy constructor std::shared_ptr<ats::io::IOSink>, when it could be moved instead.
      	Use std::move(i) instead of i.
    36    : ioSink_(i), sink_(ioSink_->branch()), sink2_(sink_->branch()), reader_(TSIOBufferReaderClone(r)), counter_(0), abort_(false)
    37  {
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficserver.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] CID 1518594: COPY_INSTEAD_OF_MOVE [trafficserver]

["maskit (via GitHub)" <gi...@apache.org>]

maskit closed issue #10344: CID 1518594: COPY_INSTEAD_OF_MOVE
URL: https://github.com/apache/trafficserver/issues/10344


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org