Opened 18 years ago

Closed 5 years ago

Last modified 4 years ago

#546 closed bug (fixed)

Unimplemented copy constructors, assignment operators

Reported by: marcusoverhagen Owned by: nobody
Priority: normal Milestone: R1/beta2
Component: - General Version: R1/alpha2
Keywords: Cc: diver
Blocked By: Blocking:
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)

etc.

Change History (7)

comment:1 by axeld, 18 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, 18 years ago

Cc: diver added

comment:3 by pulkomandy, 14 years ago

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

comment:4 by scottmc, 13 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, 7 years ago

Owner: changed from axeld to nobody
Status: newassigned

comment:6 by waddlesplash, 5 years ago

Resolution: fixed
Status: assignedclosed

Fixed in hrev53233.

comment:7 by nielx, 4 years ago

Milestone: R1R1/beta2

Assign tickets with status=closed and resolution=fixed within the R1/beta2 development window to the R1/beta2 Milestone

Note: See TracTickets for help on using tickets.