@Mr_Capehill

I would have said nothing will happen, but maybe AOS4 is different.

Otherwise one could maybe do:

yield_help_task = high pri task which does

while(1)
{
Wait(SIGBREAKF_CTRL_F);
}

my_yield()
{
Signal(yield_helper_task, SIGBREAKF_CTRL_F);
}

@Georg

Quote:

The code in the link has the WaitIO() call inside the "if(!CheckIO" block.

AARG! didn't see your semi colon, need new glasses, sorry.

OT that's one of the reasons I never use trailing clauses and *always* use braces even for single statement ifs and whiles, much much clearer to read...

As I said earlier I now think the CheckIO() is completely redundant if even [perhaps amistake. The code I use in similar circumstances looks like:

if(timerstarted)  // mustn't pass a timer that has never been started to AbortIO()
{
    AbortIO() 
    WaitIO()
    SetSignal() // if timer had finished before the WaitIO() signal won;t be cleared so clear it ourselves.
}

Last edited by broadblues on 19-Sep-2016 at 08:16 PM.

_________________
BroadBlues On Blues BroadBlues On Amiga Walker Broad

@Georg

AFAIK this should work as yield() on AmigaOS:

void amiga_yield() {
struct Task *me = FindTask(NULL);
SetTaskPri(me, me->tc_Node.ln_Pri);
}

@Mr_Capehill

Should just hog multitasking, useless code.

Last edited by NutsAboutAmiga on 19-Sep-2016 at 10:30 PM.

_________________
http://lifeofliveforit.blogspot.no/
Facebook::LiveForIt Software for AmigaOS

@Mr_Capehill

Interesting, what's the story behind that repository?

Plaz

@Georg

	thread->md.task = IDOS->CreateNewProcTags(
			NP_Entry,		threadCodeStart,
			NP_Input,		IDOS->Open("CONSOLE:", MODE_OLDFILE),
			NP_CloseInput,	TRUE,
			NP_Output,		IDOS->Open("CONSOLE:", MODE_OLDFILE),
			NP_CloseOutput,	TRUE,
			NP_StackSize,	stackSize,
			NP_Name,		thread->md.name,
			NP_EntryData,	setupMsg,
			NP_Child,		TRUE,
			NP_Cli,			TRUE,
		TAG_DONE);

	PR_ASSERT(thread->md.task != 0);

	IExec->WaitPort(setupPort);
	IExec->GetMsg(setupPort);

	IExec->FreeSysObject(ASOT_MESSAGE, setupMsg);
	IExec->FreeSysObject(ASOT_PORT, setupPort);

	return PR_SUCCESS;
}

Last edited by NutsAboutAmiga on 20-Sep-2016 at 06:01 AM.
Last edited by NutsAboutAmiga on 20-Sep-2016 at 05:59 AM.
Last edited by NutsAboutAmiga on 19-Sep-2016 at 11:07 PM.
Last edited by NutsAboutAmiga on 19-Sep-2016 at 11:03 PM.

_________________
http://lifeofliveforit.blogspot.no/
Facebook::LiveForIt Software for AmigaOS

@Deniil715

Quote:

@ferrels Please stop talking. You obviously know embarrassingly close to nothing about software development to comment on any time frames or porting effort regarding such a huge project as Firefox/Timberwolf. AmigaOS is totally alien to any other platform that porting anything bigger than a simple shell tool is a pain. The dependency tree of Firefox is monumental. I'm not claiming the Friedens are Gods, but it requires experience, talent and some serious dedication to even begin thinking about porting something like Firefox to AmigaOS, especially in the state AmigaOS was at the time. 2 (man-)years seems like a reasonable time to me. After getting 1.5 month salary for 2 years work they were probably pretty tired of the project, and then spending another month collecting the sources to make it compile in a generic environment (not only their own, most likely very custom build SDK setups) probably came very low on the priority list.

Actually I know quite a bit about software development and porting on several platforms, but I focus primarily on OS X and Windows. I've also done quite a bit of my own development over the years for OS3 and Linux as well. See my website at http://lidarwidgets.com

I also just completed a total overhaul of the software found here to bring it into the world of 64-bit computing: http://www.galiander.ca/quikgrid/availability.html

So please don't talk out of your arse any further because you know nothing about me. You're making yourself look like just another fanboy with a man-crush on the Friedens.

Last edited by ferrels on 20-Sep-2016 at 06:51 AM.
Last edited by ferrels on 20-Sep-2016 at 06:47 AM.
Last edited by ferrels on 20-Sep-2016 at 06:41 AM.

@Mr_Capehill

Who's Ronnie Sahlberg? Is him around here?

_________________
Proud Amigan since 1987
Owner of various Commodore and a SAM440ep\OS4.1FE
See them on http://retro.furinkan.org/

@Plaz

https://github.com/tfrieden/timberwolf/issues/1

@NutsAboutAmiga

It's open source, you can fix it :)

@NutsAboutAmiga

Yes that code could be tightened up WRT to error chackig but there is nothing wrong with the PR_ASSERT being there. Assuming the code you linked to (from a different project I might add!) is the correct code for this context then it simply outputs some debug if the Process failed to start and will only do so in a debug build ie when NDEBUG is not present.

To say it's totally useless except in debug mode is like saying mplayer is useless for doing anything but playing videos.... it's what its for!

Last edited by broadblues on 20-Sep-2016 at 03:37 PM.

_________________
BroadBlues On Blues BroadBlues On Amiga Walker Broad

@NutsAboutAmiga

Quote:

@Mr_Capehill Should just hog multitasking, useless code.

I assume that the Permit() might trigger some evaulation of which task should run next by the scheduler, that's not documented, so not porrtable, but the authors of the code are also the authors of the ExecSG kernel core, so I guess they ought to know.

_________________
BroadBlues On Blues BroadBlues On Amiga Walker Broad

@Mr_Capehill

I only looked at the code, I don't have Linux setup here, only run AmigaOS4.1 and Windows10, none ideal to compile timberwolf.

It's not rally a matter of fixing it works as it is sort of, but it might fail. It matter of making it safe.
It might explain way some people have no issue, and some do however. Or TW might fail in complete different place.

_________________
http://lifeofliveforit.blogspot.no/
Facebook::LiveForIt Software for AmigaOS

@broadblues

I fixed many issues like that in mplayer too, one fix in that guarantees all threads to closed, before, libraries and devices are closed. And also I coded part of threading code so stops thread before closing window and so on, when your doing things in parallel, there might be many unexpected issues, if things crash under Linux all, you get is core dump, no harm done. If you crash a AmigaOS program, you might get memory corruption and system start acting strange or lockup.

I'm not here to scrutinize the code, but just barely barely suggest what might improve it.

Last edited by NutsAboutAmiga on 20-Sep-2016 at 07:08 PM.
Last edited by NutsAboutAmiga on 20-Sep-2016 at 04:59 PM.
Last edited by NutsAboutAmiga on 20-Sep-2016 at 04:56 PM.

_________________
http://lifeofliveforit.blogspot.no/
Facebook::LiveForIt Software for AmigaOS

@NutsAboutAmiga

Yeah, well, it was a bit tongue-in-cheek comment. There is lots of talk here but little action :)

I'm still happy that some people are trying to take a constructive angle into current situation.

@salass00

Should there perhaps be a "Yield()" function in Exec? BZ time?

@NutsAboutAmiga

Quote:

I only looked at the code, I don't have Linux setup here, only run AmigaOS4.1 and Windows10, none ideal to compile timberwolf.

as i said, should be easy to set up linux crosscompiler under vmplayer on win.

@wawa

I hate wmware, I think of buying a laptop, a few times, so put away when don't use it, as does take up too much space. By the way I still have nightmare after trying to compile OWB (that was under Cygwin).

Last edited by NutsAboutAmiga on 20-Sep-2016 at 07:13 PM.
Last edited by NutsAboutAmiga on 20-Sep-2016 at 07:12 PM.

_________________
http://lifeofliveforit.blogspot.no/
Facebook::LiveForIt Software for AmigaOS

@salass00

I'm not sure about that yield thing, in ruby I used yield to inject code.

https://www.tutorialspoint.com/ruby/ruby_blocks.htm

I the reference to Javascript in this example also some kind of generator function.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/yield

I'm not sure its fully implemented maybe good idea have look at other platform's.

Last edited by NutsAboutAmiga on 20-Sep-2016 at 07:38 PM.
Last edited by NutsAboutAmiga on 20-Sep-2016 at 07:28 PM.
Last edited by NutsAboutAmiga on 20-Sep-2016 at 07:28 PM.

_________________
http://lifeofliveforit.blogspot.no/
Facebook::LiveForIt Software for AmigaOS

@NutsAboutAmiga

That's a completely different type of 'yield'

_________________
BroadBlues On Blues BroadBlues On Amiga Walker Broad