txpool_inspect: method commented out due to loop variable bug

[bomanaps]

Hi, I noticed txpool_inspect is commented out in rpc/jsonrpc/txpool_api.go (lines 205–241). Was this intentionally left out, or was it commented due to the bug in the loop where the variable is declared as txn but referenced as tx in the body?

We are working on standardizing the txpool namespace in ethereum/execution-apis#758 and txpool_inspect is one of the methods being specced. Would it be okay if I open a PR that fixes the loop bug and uncomments/completes the implementation?

[bomanaps]

cc @AskAlexSharov @JkLondon

[VBulikov]

yperbasis added Imp2 as Importance

[canepat]

In last RPC Standards call we've discussed this method and agreed to keep it out of the standard because it is very Geth-specific, see ethereum/execution-apis#758 (comment).