John Smith [ARCHIVE] on Nostr: 📅 Original date posted:2011-10-23 🗒️ Summary of this message: A scope issue ...

📅 Original date posted:2011-10-23
🗒️ Summary of this message: A scope issue in the Bitcoin code was reported by Dean Gores, but it is in a disabled feature and may not be significant.
📝 Original message:FYI
It's in a disabled feature (replacement), so I'm not sure how important this
is, but it doesn't look good.

---------- Forwarded message ----------
From: Dean Gores <dmgores at gmail.com>
Date: Sun, Oct 23, 2011 at 11:15 AM
Subject: bitcoin scope issue in main.cpp
To: laanwj at gmail.com


You have "i" and "output" declared in the same scope in "main.cpp",
AcceptToMemoryPool. Below is the offending code.

for (int i = 0; i < vin.size(); i++)
{
COutPoint outpoint = vin[i].prevout;
if (!mapNextTx.count(outpoint) ||
mapNextTx[outpoint].ptx != ptxOld)
return false;
}


Cheers,
Dean Gores
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linuxfoundation.org/pipermail/bitcoin-dev/attachments/20111023/ef8e48b1/attachment.html>;