Angus Leeming <[EMAIL PROTECTED]> writes:
| On Wednesday 06 November 2002 5:13 pm, Lars Gullik Bjønnes wrote:
>
>> use a switch:
| If I do that I get a (spurious) warning:
>
| cxx: Warning: ../../../../src/frontends/controllers/biblio.C, line 365: #117-D
| missing return statement at end of non-void function
| "biblio::searchKeys"
>
| ummm....
>
| This compiles cleanly. Ok now?
eh, no.
the default should go away.
is it enough to have the assert on its own at the bottom? but comment
why it is there.
| Angus
>
| switch (type) {
| case SIMPLE:
| {
| if (!caseSensitive)
| expr = lowercase(expr);
| SimpleMatch match(theMap,
| getVectorFromString(expr, " "),
| caseSensitive);
| return findMatch(start, keys, dir, match);
>
| }
| case REGEX:
| {
| RegexMatch const match(theMap, boost::regex(STRCONV(expr)));
| return findMatch(start, keys, dir, match);
| }
| default:
| lyx::Assert(0);
| }
| }
--
Lgb
