Update of bug #37206 (project screen):

             Assigned to:                    None => amade                  
         Planned Release:                    None => 4.7.0                  

    _______________________________________________________

Follow-up Comment #8:

Thanks for updated patch, I did some testing and it works great!

Some minor code nitpicks though:

(...)
+/* maximum mouse sequence length is SGR: ESC [ < b ; x ; y M */
+enum { MAX_MOUSE_SEQUENCE = 3+10+1+10+1+10+1 };
+
(...)
I think that this one should just be a #define

(...)
+             /* verify the mouse code (0 and 1005 are always valid) */
(...)
Um... 1005? And where it does this verification, I only see code translating
coordinates after that. Looking at older patches seems like leftover comment
from previous versions of this patch.

But yeah, I guess I can just fix those when applying. ;)

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?37206>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/


Reply via email to