Seja bem vindo ao StackOverflow @Moises ;)
Na sua função você está apenas pegando o valor do input document.calcform.visor.value
, trantando ele novoValor = valor.replace(",",".")
e atribuindo ele a variavel valor = novoValor
novamente, mas em momento nenhum está setando o valor do input com essa variavel.
Coloque isso no final de sua função e ja deve funcionar:
function substituiPonto(){
... Faz o tratamento da string aqui
document.calcform.visor.value = valor; // coloca o valor no input novamente
}
Mas há um jeito mais simples de fazer o que você precisa:
function substituir(el) {
el.value = el.value.replace(",", ".");
}
<input type="text" onkeyup="substituir(this);" />