Jump to content

Clover General discussion


ErmaC
30,171 posts in this topic

Recommended Posts

28 minutes ago, Jief_Machak said:

Does CloverUpdater and Clover use "Volumes". If no, the #ifdef __cplusplus should have them compiling again ?

Not using. Platform.h needed for <Uefi...> and for SETTINGS_DATA

Link to comment
Share on other sites

Just commited about the .h includes and gen-config.

 

I've started some includes files. The rules is :

  - each .cpp should have it's own .h

  - each .cpp should include it's own .h (to catch declaration mismatch problem)

 

So, basically, lib.h must not defines anything that are not in lib.cpp.

Platform.h could exist, but just as a convenience that includes other .h.

 

If a circular dependancy happens, it's because a module is too big...

 

 

Link to comment
Share on other sites

Also :

Ideally, a .h can include other .h, but the absolute minimum to get the type definition needed (like UINTN).

That said, Platform.h could be a candidate for this, but we have to check that Platform.h does not reference other Clover modules, especially C++ modules.

Then, gen-config should probably include Platform, and not lib.h

 

PS : I said I've started because I only moved the offending definition for gen-config. I didn't include it's own .h in every .cpp. It's a quick fix but NOT dirty. It's the right direction.

PS2 : didn't check CloverUpdater and Preference pane, but I guess it was the same problem as gen-config ?

Link to comment
Share on other sites

5 hours ago, Slice said:

I have to exclude clover-genconfig from compilation up to new times.

Hope @vector sigma will make it unnecessary.

I translated it to swift, but I'm still linking to Platform.h with no problems:

#ifdef DEBUG
#undef DEBUG
#endif

#include "../../rEFIt_UEFI/Platform/Platform.h"

#import "NSWindowFix.h"
#import "gfxutil.h"
#import "efidevp.h"

ok, not finished, but SETTINGS_DATA are ok here since last few days and the only problem until now is that I have to undefine DEBUG

Link to comment
Share on other sites

Sorry, I don't really know what gen-config does.

But in my C++ classes (foundation), I've removed the serialization method, but I have them !

 

It's base on an object XBuffer, which is almost an XArray<UINT8> in wich I've added serialize methods. Can take care of that when the moment will come... It's not that soon :-)

Link to comment
Share on other sites

28 minutes ago, vector sigma said:

 

 


cd $HOME && git clone https://github.com/CloverHackyColor/CloverBootloader.git

 

thank vector sigma I got it

 

I have another issue with Quicktime player
When i play video 1920x1080 60fps with quicktime player the screen flickering black
When i play video 1920x1080 30fps or 25fps video worked fine no flickering
What is this issue 

i think this issue is related shiki or quicktime player

I m running kabylake i5-7200u IntelHDGraphic 620

 

pr file attached

debug_21366.zip

Link to comment
Share on other sites

4 hours ago, chris1111 said:

latest build some issue but package build

 

 

 

 

Yes, I excluded genconfig compilation but not excluded a link to binary.

Will be resolved later.

  • Like 2
Link to comment
Share on other sites

Clover.app is now able to generate the config.plist and it's retro compatible with very old revisions of Clover. Anyway I set r3250 as minimum requirement as I haven't the time to go back researching even older commits. What I have to do is to add new settings that will comes in newer revisions. Old settings, if no longer present in SETTINGS_DATA, will be simply skipped because before accessing a variable in the struct the code check for its existence. So it will not crash... or at least is designed for... (ok excluding possible bugs to be fixed Lol).

885838513_Screenshot2020-03-01at15_38_35.png.88288111e144b817fdc9c396639af0e9.png

 

 

Clover.app_v1.16_r5104.pkg.zip

 

let me know!

 

P.S. now Clover.app has an installer that automatically remove attributes to prevent messages by the OS that the app is damaged due to the missing code signature of a paid certificate from Apple. Installs in /Applications, but then you can move it where you like.

10 hours ago, Slice said:

Yes, I excluded genconfig compilation but not excluded a link to binary.

Will be resolved later.

Is back. In both Clover.app and clover-genconfig now there is this definition:

#ifndef CLOVERAPPLICATION
#define CLOVERAPPLICATION 1
#endif

#include "../../rEFIt_UEFI/Platform/Platform.h"

 

in Platform.h now there's this:

#ifndef CLOVERAPPLICATION
#include "../refit/IO.h"
#endif

..should be used to exclude unneeded things.

Edited by vector sigma
  • Like 1
  • Thanks 1
Link to comment
Share on other sites

23 hours ago, anmool said:

thank vector sigma I got it

 

I have another issue with Quicktime player
When i play video 1920x1080 60fps with quicktime player the screen flickering black
When i play video 1920x1080 30fps or 25fps video worked fine no flickering
What is this issue 

i think this issue is related shiki or quicktime player

I m running kabylake i5-7200u IntelHDGraphic 620

 

pr file attached

debug_21366.zip

Sorry, I have the same IntelHDGraphic 620 in a tiny pc in my living room, ..and it flicker as well using Windows 10.

Link to comment
Share on other sites

17 hours ago, Slice said:

Yes, I excluded genconfig compilation but not excluded a link to binary.

Will be resolved later.

Sorry Slice but this time is failled :(

1891839213_Capturedcranle2020-03-0118_10_10.png.30daa49de1b8569a41a206ec7bac7354.png

 

  • Sad 1
Link to comment
Share on other sites

19 hours ago, vector sigma said:

Sorry, I have the same IntelHDGraphic 620 in a tiny pc in my living room, ..and it flicker as well using Windows 10.

I tested in windows 10 no flicker video 60fps 

this issue have in Mac intelhd620

I have a solution when I add GfxYTile patch in device property into config then flickering is gone

please test it 

I want to fix without GfxYTile patch

 

Edited by anmool
Link to comment
Share on other sites

53 minutes ago, anmool said:

I tested in windows 10 no flicker video 60fps 

this issue have in Mac intelhd620

I have a solution when I add GfxYTile patch in device property into config then flickering is gone

please test it 

I want to fix without GfxYTile patch

 

What is your config, which SMBIOS do you use! Take a look here: Piker Alpha Method   Maybe it's interesting.

Link to comment
Share on other sites

×
×
  • Create New...