You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is there any particular reason why not use this patch for both xresources and reloading it? If not I could quickly patch the thing and add this one. I think it's much simpler than the one being used right now.
The text was updated successfully, but these errors were encountered:
Rentib
changed the title
[Suggestion] Use different patch for Xresources reload.
[Suggestion] Use different patch for Xresources
Mar 4, 2024
Code wise it has everything hardcoded in the xrdb_load function and using macros to generate a bunch of repetitive code is not necessarily the most brilliant of engineering solutions, but all of it seems to be isolated so it wouldn't be out of the question to add this option alongside the other.
I haven't noticed any performance differences. It is just that the patch seems a bit simpler. I can agree that the macros aren't the best solution though.
Is there any particular reason why not use this patch for both xresources and reloading it? If not I could quickly patch the thing and add this one. I think it's much simpler than the one being used right now.
The text was updated successfully, but these errors were encountered: