4 bit 7 segment display with two 74HC595

[louislouis]

At the bottom of look: you have RETURN listed twice.

yes I forgot to delete one return, but it doesn't matter.

You turn TMR1 Off (T1CON = 0) in your Disp: Interrupt Handler but I don't see where you turn it back on.

Here: T1CON.0 = 1 ; restart timer

;---[TMR1 - interrupt handler]--------------------------------------------------
DISP:
T1CON.0 = 0                ; stop timer
TMR1H = %11111100
TMR1L = %00011111
T1CON.0 = 1                ; restart timer

Both codes which I posted works fine. I'm looking for more professional (easier, better, more skilled) example to do this.