[GitHub] guacamole-server pull request #197: GUACAMOLE-287: Added Constants in place ...

[mike-jumper]

Github user mike-jumper commented on a diff in the pull request:

    https://github.com/apache/guacamole-server/pull/197#discussion_r228670354
  
    --- Diff: src/terminal/terminal_handlers.c ---
    @@ -1422,7 +1431,7 @@ int guac_terminal_ctrl_func(guac_terminal* term, 
unsigned char c) {
         switch (c) {
     
             /* Alignment test (fill screen with E's) */
    -        case '8':
    +        case 'GUAC_TERMINAL_ALIGNMENT_TEST':
    --- End diff --
    
    This will not compile. Presumably you meant:
    
        case GUAC_TERMINAL_ALIGNMENT_TEST:
    
    and
    
        #define GUAC_TERMINAL_ALIGNMENT_TEST '8'



---