Opened 14 years ago

Closed 14 years ago

#567 closed defect (fixed)

assert(false) in EnsembleBuilder::test_data(const KernelLookup&, size_t)

Reported by: Peter Owned by: Peter
Priority: major Milestone: yat 0.5.6
Component: classifier Version: 0.5.4
Keywords: Cc:

Description

There should be no assert i header files.

Why is there an assert(false)? Does it imply that the code is useless or can we simply remove it?

Change History (4)

comment:1 Changed 14 years ago by Peter

(In [2077]) refs #567 using YAT_ASSERT in header rather than assert

comment:2 Changed 14 years ago by Peter

Status: new → assigned

The assert appears in r1206 and it seems as it is not needed, so I think we can remove. I want to add a test, however, that uses the code in question before closing this ticket.

comment:3 Changed 14 years ago by Peter

Component: test → classifier
Milestone: yat 0.6 → yat 0.5.6
Version: trunk → 0.5.4

comment:4 Changed 14 years ago by Peter

Resolution: → fixed
Status: assigned → closed

(In [2133]) fixes #567

Note: See TracTickets for help on using tickets.