[GitHub] [trafficserver] bneradt commented on a change in pull request #7553: Get rid of kruft LogObject copy constructor.

[GitBox <gi...@apache.org>]

bneradt commented on a change in pull request #7553:
URL: https://github.com/apache/trafficserver/pull/7553#discussion_r582089737



##########
File path: proxy/logging/LogObject.h
##########
@@ -97,7 +97,6 @@ class LogObject : public RefCountObj
             const char *header, Log::RollingEnabledValues rolling_enabled, int flush_threads, int rolling_interval_sec = 0,
             int rolling_offset_hr = 0, int rolling_size_mb = 0, bool auto_created = false, int rolling_max_count = 0,
             int rolling_min_count = 0, bool reopen_after_rolling = false, int pipe_buffer_size = 0);
-  LogObject(LogObject &);

Review comment:
       We should probably delete this and the copy assignment constructors to make sure they don't accidentally get used in the future.
   
   > LogObject(LogObject const&) = delete;
   > LogObject& operator=(LogObject const&) = delete;




----------------------------------------------------------------
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.

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