Calling SAID-CRUMBS? from EBAG doesn't work as intended

[eriktorbjorn]

EBAG-F has this, which is apparently there to check if you refer to the empty (or soggy) bag as "crumbs":

<ROUTINE EBAG-F ()
     <COND (<SAID-CRUMBS?>
        <TELL "The crumbs are all gone." CR>
        <RFATAL>)

But it doesn't work. Probably because the first thing SAID-CRUMBS? does is to check if you're carrying the full bag of crumbs (BAG):

<ROUTINE SAID-CRUMBS? ()
     <COND (<NOT <IN? ,BAG ,PLAYER>>
        <RFALSE>)

[eriktorbjorn]

Similarly, the CRUMBS-GONE? routine will only work if you are carrying the bag of crumbs, even though it's clearly meant to work if you are carrying the empty or soggy bag as well:

<ROUTINE CRUMBS-GONE? ()
     <COND (<NOT <SAID-CRUMBS?>>
        <RFALSE>)
           (<PRSO? BAG>
        <TAKE-CRUMBS>
        <RTRUE>)
           (<PRSO? EBAG SOGGY>
        <TELL "The crumbs are all gone." CR>
        <RTRUE>)>
     <RFALSE>>

This routine is called from:

• PRE-GIVE
• PRE-PUT
• PRE-THROW
• PRE-TAKE

[eriktorbjorn]

The SAID-CRUMBS? routine is called from:

• GWIM, where it already checks if you mean the BAG object.
• BAG-F, so that should already be the BAG object.
• EBAG-F, as described above.
• CRUMBS-GONE?, as described above.

So I guess it should be safe to remove that check from SAID-CRUMBS?? Or did I miss something?