Login
Username:

Password:

Remember me



Lost Password?

Register now!

Sections

Who's Online
52 user(s) are online (34 user(s) are browsing Forums)

Members: 1
Guests: 51

mdr, more...

Headlines

 
  Register To Post
« 1 2 (3) 4 5 »

Re: SpotLess tests need it to find out one bug
Quite a regular
Quite a regular


See User information
@alfkil

Did a fast test and when quitting I get:

***¡Command 'KDH9:SDK/Spotless20220516' returned mising 'Disable'!

Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@jabirulo

And now? Spotless

Go to top
Re: SpotLess tests need it to find out one bug
Quite a regular
Quite a regular


See User information
@alfkil
Same

#Devel:SDK/Spotless
Hello.

***¡Command 'Devel:SDK/Spotless' returned missing 'Disable'!

...
int main() {
    
puts("Hello.\n");
    
struct DIPrefs test;
    
trial(&test);
    return 
0;
}
#


Set BP in puts("Hel...) and trial(&test) got same (above) result when quitting.

Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@jabirulo

Hmm. Strange. I am not sure, I can fix that without having a machine to run it.

Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@jabiruolo

It is really strange, because I have commented out every call to Enable() and Disable(). I will need a working machine to test.

Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@jabirulo

Just a quick question : Are you letting the child finish before exiting?

Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@kas1e

Quote:
EDIT: Also found GUI bug, to reproduce: -- run spotless -- hit RMB , choose "configure" -- see, on the left/top side, you have some empty small square appears and stay there since that time forever.


Could you post a video or screenshot of this?

Go to top
Re: SpotLess tests need it to find out one bug
Quite a regular
Quite a regular


See User information
@alfkil

I just put 2 BP, Start button (1st BP) and then "Step over" (I'm on 2nd BP) and I click close gadget, I don't let main() to finish (reach 'return 0').

int main() {
    
puts("Hello.\n");   // 1st BP
    
struct DIPrefs test;
    
trial(&test);       // 2nd BP
    
return 0;
}
...
void trial(struct DIPrefs *diprefs) {
    
diprefs 0;
    return;
}

Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@jabirulo

Then that is the reason. If you don't let the child finish, there will be complaints from the system. I cannot change this. The only thing I could do would be to prevent you from quiting Spotless. And I don't want to do that.

So : The reason for the error is, that you have no way to force close a child in AmigaOS.

Go to top
Re: SpotLess tests need it to find out one bug
Quite a regular
Quite a regular


See User information
@alfkil

Ok, without knowing Spotless (and system) internals, why Enable/Disable? Couldn't be changed to use Semaphore/Mutex and on quitting just "release"'em?

Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@jabirulo

In the last version you ran, there are no references in the code to Enable()/Disable(). So the relevant calls must be local to the system, or the error message is misleading.

Go to top
Re: SpotLess tests need it to find out one bug
Home away from home
Home away from home


See User information
@alfkil
Quote:

can you post a video or screenshot of it?


Sure, there is:

https://youtu.be/hpnqyJLxINs

See, i just run congirue, and it reate some strange gadget at bottom left area of spotless window. This one never disappear and dunno what the purposes of it are. Feels like just GUI bug, imho.

Go to top
Re: SpotLess tests need it to find out one bug
Quite a regular
Quite a regular


See User information
@alfkil

ok, thx for explanation.

Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
Having a strange gadget on corners of a window means the program code is using a wrong variant of Set...Attrs functions or not passing the window pointer. Some gadgets require their own variants to work corretly like Clicktab and Virtual.

Rock lobster bit me, so excuse me.
X1000 + AmigaOS 4.1 FE
"Anyone can build a fast CPU. The trick is to build a fast system." - Seymour Cray
Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@TSK

Thanks!

@kas1e

Does the same problem appear with the MemorySurfer?

Go to top
Re: SpotLess tests need it to find out one bug
Home away from home
Home away from home


See User information
@alfkil
Quote:

Does the same problem appear with the MemorySurfer?


When i just click on memory surfer gadget ? No, all fine.

Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@TSK and kas1e

I have replaced

Layout *Layout::createLabeledLayout (string label) {
     
Layout *childLayout = new Layout(parentlabel);
     
IIntuition->SetAttrs (layout,
         
LAYOUT_AddChildchildLayout->systemObject(),
     
TAG_DONE);
     return 
childLayout;
}


with

Layout *Layout::createLabeledLayout (string label) {
     
Layout *childLayout = new Layout(parentlabel);
     
IIntuition->SetGadgetAttrs (layoutparent->windowPointer(), 0,
         
LAYOUT_AddChildchildLayout->systemObject(),
     
TAG_DONE);
     return 
childLayout
}


Does this improve things?

Spotless_SetGadgetAttrs


Edited by alfkil on 2022/5/20 16:03:19
Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@thread

Sorry, found the bug. This one should fix it :

Spotless_parent_fix

Go to top
Re: SpotLess tests need it to find out one bug
Home away from home
Home away from home


See User information
@alfkil
Quote:

Sorry, found the bug. This one should fix it :

Spotless_parent_fix


Tested. Cosmetic bug with "configure" in GUI gone.
Also defaul FPR registers looks sane after loading.

But i do find, that last entry in the registers are "trash", it show 0 and rectangle box. Like, you forget to terminate strings massive or something of that sort (maybe mislead \0 with just 0 or something).

Also "Step out" crash bug i mention seems gone, but not sure if it reacts as expected , now that what i do:

1. run spotless
2. load or test case
3. hit bp on "puts"
4. once it breaks, hit "step out"

At this point test case exit seems so, or something , but i do not have "Hello" in console.

Go to top
Re: SpotLess tests need it to find out one bug
Just can't stay away
Just can't stay away


See User information
@kas1e

Quote:
But i do find, that last entry in the registers are "trash", it show 0 and rectangle box. Like, you forget to terminate strings massive or something of that sort (maybe mislead \0 with just 0 or something).


Screenshot?

EDIT : This is the responsible code :

result.push_back(printStringFormat("%" PRIu64 "\n", (uint64_t)context->fpscr));


I am not sure, if this is the correct way to print a 64-bit integer. I just copied it from some internet page. Does anyone have a good take on this?

Quote:
Also "Step out" crash bug i mention seems gone, but not sure if it reacts as expected , now that what i do:
1. run spotless
2. load or test case
3. hit bp on "puts"
4. once it breaks, hit "step out"

At this point test case exit seems so, or something , but i do not have "Hello" in console.


Are you sure?? Hello should come in the shell window of the caller (where you ignite Spotless). In any case, try this and see if it helps :

Spotless_isFunction_in_stepOut


Edited by alfkil on 2022/5/20 18:46:36
Go to top

  Register To Post
« 1 2 (3) 4 5 »

 




Currently Active Users Viewing This Thread: 1 ( 0 members and 1 Anonymous Users )




Powered by XOOPS 2.0 © 2001-2016 The XOOPS Project