<br><br><div><span class="gmail_quote">2006/4/14, Dan Winship <<a href="mailto:danw@novell.com">danw@novell.com</a>>:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
reggaemanu wrote:<br>> Here is a patch to fix the segfault issue with command_screenshot and<br>> window_command_screenshot.<br><br>Er? What segfault issue? Was this brought up somewhere else?</blockquote><div><br>Yes, on some forums/irc, try it...nobody can use the Screenshot command, that definitely broke compiz.
<br>And for sure...look the code carefuly :<br>for the moment...Print try to run COMP_DISPLAY_OPTION_RUN_COMMAND_SCREENSHOT instead of COMP_DISPLAY_OPTION_COMMAND_SCREENSHOT, but it's the shortcut keys, not the command (same problem for <Alt>Print)...so this should never work and compiz doesn't like that...
<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">> It also avoid the conflict between this two functions if both contain<br>
> the same key (Print and <Alt>Print for example)<br>> (actually if you press <Alt>Print both commands are triggered)<br><br>Hm... but it just switches the problem around, right? So that if you<br>swapped the bindings so that "Print" was "run_command_window_screenshot"
<br>and "<Alt>Print" was "run_command_screenshot", then "<Alt>Print" would<br>trigger both of them? To really fix it, it needs to be checking that<br>there are no extraneous modifiers held down. Then it can check the
<br>bindings in any order and it will work, regardless of what keys are<br>bound to what.</blockquote><div><br>Hum, now you say that, yes you are right, it's just a (temporary !?) fix for the defaults shortcut, but at least it's better than the actual code since you have one possibility to use both of shortcut (actualy even if we fix the above problem we definitely can't), i will take a quick look to see if I can do it better.
<br>Anyway, i'm not a coder and don't know anything about C language, so if you can do it in a better way...just do it ;)<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
-- Dan<br></blockquote></div><br>