Closed BlaiseRideout closed 13 years ago
Hey there, dude. Nice to see you managed to get it working how you wanted it to.
A couple of issues with the merge that I'll look at dealing with later: -- The update breaks Xinerama support (should be an easy fix) -- Obviously needs the changes as optional flags and documentation (also pretty easy) -- 100% CPU usage while running (may be unfixable but could be counterbalanced by a warning in usage instructions)
I've got to work until later (I'm on Britfag time) but I've pulled your changes locally and I'll look at doing a manual merge/reshuffling when I can. If you do anything to fix these issues then let me know (and I'll do the same in return) so that we don't end up duplicating effort.
When I try it (I took out the fullscreen tag, this might change things) I only get 20-25% CPU usage. I'll get the optional flags and documentation, I don't know anything about the Xinerama support, so I'll let you handle that one
I didn't make it an option, and didn't make it very universal, sorry about that. I'm the guy on /g/ who wanted the cat to scale to the width of the screen. I also don't have your most recent commit updates, the ones for the memory error handling. This is mostly proof of concept
[EDIT] Added the memory error handling