Hie
I'm a newbie in pic programming and i'm trying to build a night lamp with 30 leds. at start they are all supposed to be lit and then after every minute one of them should turn off.
I built the circuit using 4 74hc595 cascaded and with a pic16f684 i had laying around with a 4 MHZ xtal.
my question is about the code - if i understand correctly, the code it should look like this:

Include "modedefs.bas" ' Include shift modes

D1PIN var PORTA.0 ' Shift data pin 1
C1PIN var PORTA.1 ' Shift clock pin 1

bvar var BYTE

bvar = 255
' Shift out 8 bits of data onto other pins
Shiftout D1PIN, C1PIN, MSBFIRST,[bvar]
Shiftout D1PIN, C1PIN, MSBFIRST,[bvar]
Shiftout D1PIN, C1PIN, MSBFIRST,[bvar]
Shiftout D1PIN, C1PIN, MSBFIRST,[bvar]

Main:
sleep 1000
Shiftout D1PIN, C1PIN, MSBFIRST,[0]
goto Main
end

Is this ok? im getting the programmer ready now and i just wanted to get some input before i actually program it because its my first project.

Thanks in advance,
Shahar Zrihen.