[Bug 469] New: Copy constructor of DiscretePdf

For more infomation about this bug, visit
Summary: Copy constructor of DiscretePdf
Product: BFL
Version: trunk
Platform: All
OS/Version: All
Status: NEW
Severity: normal
Priority: P2
Component: core
AssignedTo: bfl [..] ...
ReportedBy: tinne [dot] delaet [..] ...
CC: bfl [..] ...
Estimated Hours: 0.0

The copy constructor of the discretepdf contains some errors according to me.
First of all, the dimension of the pdf is not set.
Furthermore, the Cumulative pdf is not updated.

Tinne

[Bug 469] Copy constructor of DiscretePdf

For more infomation about this bug, visit

--- Comment #5 from Tinne De Laet <tinne [dot] delaet [..] ...> 2007-12-20 08:54:39 ---
I applied your improved patch in revision 28792.

Sending src/pdf/discretepdf.cpp
Sending src/pdf/pdf.h
Transmitting file data ..
Committed revision 28792.

[Bug 469] Copy constructor of DiscretePdf

For more infomation about this bug, visit

--- Comment #4 from Klaas Gadeyne <klaas [dot] gadeyne [..] ...> 2007-12-19 17:10:14 ---
Created an attachment (id=169)
--> (https://www.fmtc.be/bugzilla/orocos/attachment.cgi?id=169)
Better patch (IMHO)

I think this solution is somewhat better: it will keep working if someone
changes the internals of the Pdf class (which might be the case sooner or later
:-)

[Bug 469] Copy constructor of DiscretePdf

For more infomation about this bug, visit

Tinne De Laet <tinne [dot] delaet [..] ...> changed:

What |Removed |Added
--------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED

--- Comment #3 from Tinne De Laet <tinne [dot] delaet [..] ...> 2007-12-19 16:48:11 ---
Patch applied in revision 28791.

Sending src/pdf/discretepdf.cpp
Sending src/pdf/pdf.h
Sending tests/pdf_test.cpp
Transmitting file data ...
Committed revision 28791.

[Bug 469] Copy constructor of DiscretePdf

For more infomation about this bug, visit

--- Comment #2 from Tinne De Laet <tinne [dot] delaet [..] ...> 2007-12-19 16:23:59 ---
Created an attachment (id=168)
--> (https://www.fmtc.be/bugzilla/orocos/attachment.cgi?id=168)
Patch which IMHO solves the bug

[Bug 469] Copy constructor of DiscretePdf

For more infomation about this bug, visit

Tinne De Laet <tinne [dot] delaet [..] ...> changed:

What |Removed |Added
--------------------------------------------------------------------------
Status|NEW |ASSIGNED

--- Comment #1 from Tinne De Laet <tinne [dot] delaet [..] ...> 2007-12-19 16:17:16 ---
I'm trying to solve this.