That syntax should work OK, but you can try this alternative.
<a href="https://stackoverflow.com/questions/5003867/javascript:void(0);" onclick="ShowOld(2367,146986,2);">
or
<a href="https://stackoverflow.com/questions/5003867/javascript:ShowOld(2367, 146986, 2);">
UPDATED ANSWER FOR STRING VALUES
If you are passing strings, use single quotes for your function’s parameters
<a href="https://stackoverflow.com/questions/5003867/javascript:ShowOld("foo', 146986, 'bar');">