maxQuoteQuantity is being set incorrectly for bids

[sconybeare]

maxQuoteQuantity is being calculated as price*quantity, which fails to take fees into account when the order is a bid that matches against a resting offer. Instead, maxQuoteQuantity should be set to price*quantity*(1+fee_rate), unless the order is post-only.

[yamen]

Background on this is here: https://docs.google.com/document/d/17-FlYTIcLiciWdjXaQEdUnEJFbvIeIEYW3fdcEQu8ds/edit

Currently a brute force approach of using 1% is done when creating the wrapped SOL account in cases where quote token == SOL:

serum-ts/packages/serum/src/market.ts

Line 672 in 5f7ceaf

lamports = Math.round(price * size * 1.01 * LAMPORTS_PER_SOL);

Suggesting the same could be done here for consistency:

serum-ts/packages/serum/src/market.ts

Lines 817 to 818 in 5f7ceaf

	maxQuoteQuantity: new BN(this._decoded.quoteLotSize.toNumber()).mul(
	this.baseSizeNumberToLots(size).mul(this.priceNumberToLots(price)),

Ideal version of course takes into account the actual fees expected, but given there are scenarios where it's not obvious if order will be a taker or maker, this is not foolproof.

Alternatively the current PR (#86) fixes the issue properly by assuming taker fees as long as its not a postOnly order.