[LUA] Problems with a boot loader...

function start1()
--- Do stuff

end
function time()
timer()
end
parallel.waitForAny(start1, time)

Mikk809h, on 07 January 2013 - 11:32 AM said:

Your term.cleaLine(6,14) should be
term.clearLine() – Without Any arguments.

Your term.clearLine(6,14) is throwing an error, which shuts down the system, before you should know it.
and you should call your parallel.waitForAny() after the two functions :
Like :


function() – make a name for the function. Like:

function start1()
--- Do stuff

end
function time()
timer()
end
parallel.waitForAny(start1, time)

I think that would solve the problem :P/>


:D/>

Mikk809h, on 07 January 2013 - 11:32 AM said:

Your term.cleaLine(6,14) should be
term.clearLine() – Without Any arguments.

Your term.clearLine(6,14) is throwing an error, which shuts down the system, before you should know it.
and you should call your parallel.waitForAny() after the two functions :
Like :


function() – make a name for the function. Like:

function start1()
--- Do stuff

end
function time()
timer()
end
parallel.waitForAny(start1, time)

I think that would solve the problem :P/>


:D/>

Orwell, on 07 January 2013 - 11:40 AM said:

Mikk809h, on 07 January 2013 - 11:32 AM said:

Your term.cleaLine(6,14) should be
term.clearLine() – Without Any arguments.

Your term.clearLine(6,14) is throwing an error, which shuts down the system, before you should know it.
and you should call your parallel.waitForAny() after the two functions :
Like :


function() – make a name for the function. Like:

function start1()
--- Do stuff

end
function time()
timer()
end
parallel.waitForAny(start1, time)

I think that would solve the problem :P/>


:D/>

You're wrong twice. You can perfectly call term.clearLine(6,14), it will just ignore the parameters. You can also perfectly define the functions within the call to parallel.waitForAny. Thus the problem is something different. If no one finds it soon, I will take a look myself.

Mikk809h, on 07 January 2013 - 07:35 PM said:

And the parallel: i just think that it would be easier to just make two words in the paratheses.. ::-/
sorry.

-- Made by remiX for Util OS
selection = 1

t_Options = {
    { text = "Util OS 0.1 BETA", handler = function() shell.run(".Util/os") end },
    { text = os.version(), handler = function() term.setBackgroundColour(colours.black) term.setTextColour(colours.white) term.setCursorPos(1, 1) term.clear() end }
}

function printMain()
    term.setBackgroundColour(colours.black)
    term.setTextColour(colours.white)
    term.clear()
    term.setCursorPos(1, 2)
    print(" +------------------Boot Loader------------------+ ")
    print(" |											   | ")
    print(" +-----------------------------------------------+ ")
    print(" |											   | ")
    print(" |											   | ")
    print(" |											   | ")
    print(" |											   | ")
    print(" |											   | ")
    print(" |											   | ")
    print(" |											   | ")
    print(" |											   | ")
    print(" |											   | ")
    print(" +-----------------------------------------------+ ")
    print(" |											   | ")
    print(" |											   | ")
    print(" |											   | ")
    print(" +-----------------------------------------------+ ")
end

function timer()
    for i = 5, 1, -1 do
        term.setBackgroundColour(colours.black)
        term.setCursorPos(6, 16)
        write("Booting in " .. i)
        sleep(1)
    end
    return
end

function text()
    while true do
        for op = 1, #t_Options do
            term.setCursorPos(7, op + 5)
            if op == selection then term.setBackgroundColour(colours.lightGrey) else term.setBackgroundColour(colours.black) end
            write(t_Options[op].text)
        end
        local _, key = os.pullEvent("key")
        if key == keys.down and selection < #t_Options then
            -- Down
            selection = selection + 1
        elseif key == keys.up and selection > 1 then
            -- Up
            selection = selection - 1
        elseif key == keys.enter then
            t_Options[selection].handler()
            return
        end
    end
end

printMain()
parallel.waitForAny(timer, text)

-- This is here and will only occur if the timer runs out
-- Because of the sleep(2) (so you can read what it is booting)
-- I had to put it here or else you could still use up and down keys
-- while it's booting
printMain()
term.setCursorPos(7, 5)
print("BOOTING...")
term.setCursorPos(7, 6)
write(t_Options[selection].text)
sleep(2)
t_Options[selection].handler()

Mikk809h, on 07 January 2013 - 07:35 PM said:

Oh then im really sorry, for posting wrong info…
i didnt really know that it would ignore the parametres.
And the parallel: i just think that it would be easier to just make two words in the paratheses.. ::-/
sorry.

remiX, on 07 January 2013 - 09:23 PM said:

Mikk809h, on 07 January 2013 - 07:35 PM said:

And the parallel: i just think that it would be easier to just make two words in the paratheses.. ::-/
sorry.

It is but it's really up to the coder to do what he wants :)/>

I'm busy looking it at it now and I see you call text() within text() within the parallel.waitForAny(). Will that not give a stack overflow at times, or maybe the time will run out in time..

edit: I think the problem does lie within the parallel function, where you call text() all the time. Weird that the time keeps changing seconds, maybe you should have used a for loop for the timer function.

Also, if you don't know about Tables, you should learn about them and use them a lot, they are really useful.

Check this out:

Spoiler

-- Made by remiX for Util OS
selection = 1

t_Options = {
	{ text = "Util OS 0.1 BETA", handler = function() shell.run(".Util/os") end },
	{ text = os.version(), handler = function() term.setBackgroundColour(colours.black) term.setTextColour(colours.white) term.setCursorPos(1, 1) term.clear() end }
}

function printMain()
	term.setBackgroundColour(colours.black)
	term.setTextColour(colours.white)
	term.clear()
	term.setCursorPos(1, 2)
	print(" +------------------Boot Loader------------------+ ")
	print(" |											   | ")
	print(" +-----------------------------------------------+ ")
	print(" |											   | ")
	print(" |											   | ")
	print(" |											   | ")
	print(" |											   | ")
	print(" |											   | ")
	print(" |											   | ")
	print(" |											   | ")
	print(" |											   | ")
	print(" |											   | ")
	print(" +-----------------------------------------------+ ")
	print(" |											   | ")
	print(" |											   | ")
	print(" |											   | ")
	print(" +-----------------------------------------------+ ")
end

function timer()
	for i = 5, 1, -1 do
		term.setBackgroundColour(colours.black)
		term.setCursorPos(6, 16)
		write("Booting in " .. i)
		sleep(1)
	end
	return
end

function text()
	while true do
		for op = 1, #t_Options do
			term.setCursorPos(7, op + 5)
			if op == selection then term.setBackgroundColour(colours.lightGrey) else term.setBackgroundColour(colours.black) end
			write(t_Options[op].text)
		end
		local _, key = os.pullEvent("key")
		if key == keys.down and selection < #t_Options then
			-- Down
			selection = selection + 1
		elseif key == keys.up and selection > 1 then
			-- Up
			selection = selection - 1
		elseif key == keys.enter then
			t_Options[selection].handler()
			return
		end
	end
end

printMain()
parallel.waitForAny(timer, text)

-- This is here and will only occur if the timer runs out
-- Because of the sleep(2) (so you can read what it is booting)
-- I had to put it here or else you could still use up and down keys
-- while it's booting
printMain()
term.setCursorPos(7, 5)
print("BOOTING...")
term.setCursorPos(7, 6)
write(t_Options[selection].text)
sleep(2)
t_Options[selection].handler()

Oct125, on 08 January 2013 - 05:52 AM said:

–snip

Thanks! This will help a lot! I'll put your name on the credits session ;)/>

And i modified the code a little to, for example, bypass the boot loader if a function in my os it is activated…