feat(primitives): remove TxValue wrapper

[shekhirin]

U256 uses a compact implementation when encoding/decoding to/from database, so there's no difference between explicit u128 and U256 types. We can remove the Ethereum/Optimism branching logic from the TxValue wrapper alongside the wrapper itself.

reth/crates/primitives/src/transaction/tx_value.rs

Lines 44 to 78 in d2f9c4f

	/// As ethereum circulation on mainnet is around 120mil eth as of 2022 that is around
	/// 120000000000000000000000000 wei we are safe to use u128 for TxValue's encoding
	/// as its max number is 340282366920938463463374607431768211455.
	/// This optimization should be disabled for chains such as Optimism, where
	/// some tx values may require more than 128-bit precision.
	impl Compact for TxValue {
	#[inline]
	fn to_compact<B>(self, buf: &mut B) -> usize
	where
	B: bytes::BufMut + AsMut<[u8]>,
	{
	#[cfg(feature = "optimism")]
	{
	self.0.to_compact(buf)
	}
	#[cfg(not(feature = "optimism"))]
	{
	self.0.to::<u128>().to_compact(buf)
	}
	}
	#[inline]
	fn from_compact(buf: &[u8], identifier: usize) -> (Self, &[u8]) {
	#[cfg(feature = "optimism")]
	{
	let (i, buf) = U256::from_compact(buf, identifier);
	(TxValue(i), buf)
	}
	#[cfg(not(feature = "optimism"))]
	{
	let (i, buf) = u128::from_compact(buf, identifier);
	(TxValue::from(i), buf)
	}
	}
	}

[mattsse]

lgtm