Opened 13 years ago

Closed 8 weeks ago

#546 closed bug (fixed)

Unimplemented copy constructors, assignment operators

Reported by: marcusoverhagen Owned by: nobody
Priority: normal Milestone: R1
Component: - General Version: R1/alpha2
Keywords: Cc: diver
Blocked By: Blocking:
Has a Patch: no Platform: All

Description (last modified by pulkomandy)

The assignment operator and copy constructor should be unimplemented, thats the whole reason for them beeing private.

Please remove implementations like

BLooper::BLooper(const BLooper&)
BLooper& BLooper::operator=(const BLooper& )
BHandler::BHandler(const BHandler &)
BHandler & BHandler::operator=(const BHandler &)
BApplication::BApplication(const BApplication &rhs)
BApplication & BApplication::operator=(const BApplication &rhs)


Change History (6)

comment:1 by axeld, 13 years ago

I agree with you that it's not really useful, but since R5 exports those methods, we might want to keep them around (even if #ifdef'd out).

comment:2 by diver, 13 years ago

Cc: diver added

comment:3 by pulkomandy, 9 years ago

Description: modified (diff)
Platform: All
Version: R1/alpha2

comment:4 by scottmc, 9 years ago

So what's the task here? To ifdef these out? Or has that already been done in the 5 years since this ticket was opened?

comment:5 by axeld, 2 years ago

Owner: changed from axeld to nobody
Status: newassigned

comment:6 by waddlesplash, 8 weeks ago

Resolution: fixed
Status: assignedclosed

Fixed in hrev53233.

Note: See TracTickets for help on using tickets.