At a minimum, help text should note this. This is true for things like addwitnessaddress especially.
bool p2sh = true;
    if (!request.params[1].isNull()) {
        p2sh = request.params[1].get_bool();
    }
    Witnessifier w(pwallet);
    bool ret = boost::apply_visitor(w, dest);
    if (!ret) {
        throw JSONRPCError(RPC_WALLET_ERROR, "Public key or redeemscript not known to wallet, or the key is uncompressed");
    }
    CScript witprogram = GetScriptForDestination(w.result);
    if (p2sh) {
        w.result = CScriptID(witprogram);
    }
    if (w.already_witness) {
        if (!(dest == w.result)) {
            throw JSONRPCError(RPC_WALLET_ERROR, "Cannot convert between witness address types");
        }
    } else {
        pwallet->AddCScript(witprogram);
        pwallet->SetAddressBook(w.result, "", "receive");
    }
    return EncodeDestination(w.result);
}
struct tallyitem
{
    CAmount nAmount;
    int nConf;
    std::vector<uint256> txids;
    bool fIsWatchonly;
    tallyitem()