12 replies to this topic

[KaoS]

Hey pro people, I am busy making a screen logger and resumer for a project of mine. everything is working fine except it will not take the term.scroll function into account.

I apologise that it is such a long code, the scroll modifier starts at line 104 but you may need to read the top part to understand how the record works, here is the pastebin: http://pastebin.com/JMUMUgUL

Thanks for the help

[KaoS]

no ideas guys? I added comments to the code to make it a little easier to understand, in order to fully understand it you need to understand metatables, before I understood them I tried Google and found an amazing article. in case you need it here is a link

[ChunLing]

Sorry, I'm fixing my code to get rid of ipairs (and do other stuff). This function still successfully calls the oterm.scroll, right? It just doesn't log correctly all the information you're trying to save. Which...are you trying to save the entire screen?

[KaoS]

exactly. as you write to your screen it writes to my log. it works perfectly except it doesn't scroll

[PixelToast]

mine just attaches perimeter hooks to all the term functions :3

[ChunLing]

Why are you trying to save the whole screen? The log already contains all the output information, you just need to log that the screen scrolled, not reiterate everything that was on the screen when it scrolled.

[KaoS]

I am moving everything up one in the log. it doesn't need everything ever entered, just what is currently on the screen

[ChunLing]

Looks more complicated than necessary. I have to let my brain recover a while before tackling it.

[Lyqyd]

You are doing this the hard way, you know. Why not just use a redirect target that updates a buffer table before calling the main term functions?

[KaoS]

I'm not sure how that works... anyways thanks for the help. I'll take a look

[Lyqyd]

An example of this would be found near the top of this file: https://github.com/l...b/master/window

That may be a useful reference or source of inspiration.

[KaoS]

Thanks, I am looking through it. the thing is I cannot find one instance of you actually calling the function new(). trying to figure out the syntax for target. guess I'll just look up the full code

but great code hey

EDIT: found it, I was thinking of doing it this way. I tried it in the past and made a mess of it. oh well, back to the drawing board /> time for an overhaul. thanks for the help Lyqyd, you too ChunLing. much appreciated

[KaoS]

lol still not working. I have a question. term.native.write ignores redirects right?