On Sat, Apr 25, 2009 at 03:17:17PM +1000, Brian May wrote:
> So if I understand this patch correctly, set_active_frame uses the
> global variable rp_current_screen to determine what screen the frame
> belongs to, which may be wrong, so this is a hack to "update" the
> current screen so it is "correct" for the frame.

On second thoughts, it looks like set_active_frame already does the
right thing:

void
set_active_frame (rp_frame *frame, int force_indicator)
{
  rp_screen *old_s = current_screen();
  rp_screen *s = frames_screen (frame);

[...]

So either I misunderstood the patch, or it does something that has
already been fixed.
-- 
Brian May <[email protected]>


_______________________________________________
Ratpoison-devel mailing list
[email protected]
http://lists.nongnu.org/mailman/listinfo/ratpoison-devel

Reply via email to