Hey guys, will this work?

[mister_e]

The change is above... <<4 instead of <<5... but as i said... <<5 was wrong anyway... not sure why, but wrong anyways... few years after... no one complaint

It is more code efficient... and obviously safer to use
CCPR1L = duty>>2
CCP1CON.4=Duty.0
CCP1CON.5=Duty.1

unless you'll screw up the CCP module on that 18F (think about the bit <7:6> of CCP1CON when duty value change). This was my major point here.