Changeset dc0f6e4c in mainline


Ignore:
Timestamp:
2023-10-24T16:50:46Z (7 months ago)
Author:
Jiří Zárevúcky <zarevucky.jiri@…>
Parents:
29ed06d
git-author:
Jiří Zárevúcky <zarevucky.jiri@…> (2023-10-22 20:38:41)
git-committer:
Jiří Zárevúcky <zarevucky.jiri@…> (2023-10-24 16:50:46)
Message:

Deduplicate identical enums for window resizing

Location:
uspace/lib
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • uspace/lib/display/include/types/display/wndresize.h

    r29ed06d rdc0f6e4c  
    3838/** Window resize type */
    3939typedef enum {
     40        display_wr_none = 0,
     41
    4042        display_wr_top = 0x1,
    4143        display_wr_left = 0x2,

  • uspace/lib/display/src/wndresize.c

    r29ed06d rdc0f6e4c  
    7575bool display_wndrsz_valid(display_wnd_rsztype_t rsztype)
    7676{
    77         bool valid;
    78 
    79         valid = false;
    80 
    8177        switch (rsztype) {
    8278        case display_wr_top:
    … …  
    8884        case display_wr_top_right:
    8985        case display_wr_bottom_left:
    90                 valid = true;
    91                 break;
     86                return true;
     87
     88        case display_wr_none:
     89                return false;
    9290        }
    9391
    94         return valid;
     92        return false;
    9593}
    9694

  • uspace/lib/ui/include/types/ui/wdecor.h

    r29ed06d rdc0f6e4c  
    4040#include <types/common.h>
    4141#include <types/ui/cursor.h>
     42#include <types/display/wndresize.h>
    4243
    4344struct ui_wdecor;
    … …  
    6768} ui_wdecor_style_t;
    6869
    69 /** Window resize type */
    70 typedef enum {
    71         ui_wr_none = 0,
    72 
    73         ui_wr_top = 0x1,
    74         ui_wr_left = 0x2,
    75         ui_wr_bottom = 0x4,
    76         ui_wr_right = 0x8,
    77 
    78         ui_wr_top_left = ui_wr_top | ui_wr_left,
    79         ui_wr_bottom_left = ui_wr_bottom | ui_wr_left,
    80         ui_wr_bottom_right = ui_wr_bottom | ui_wr_right,
    81         ui_wr_top_right = ui_wr_top | ui_wr_right
    82 } ui_wdecor_rsztype_t;
    83 
    8470/** Window decoration callbacks */
    8571typedef struct ui_wdecor_cb {
    … …  
    9379        void (*close)(ui_wdecor_t *, void *);
    9480        void (*move)(ui_wdecor_t *, void *, gfx_coord2_t *, sysarg_t);
    95         void (*resize)(ui_wdecor_t *, void *, ui_wdecor_rsztype_t,
     81        void (*resize)(ui_wdecor_t *, void *, display_wnd_rsztype_t,
    9682            gfx_coord2_t *, sysarg_t);
    9783        void (*set_cursor)(ui_wdecor_t *, void *, ui_stock_cursor_t);

  • uspace/lib/ui/private/wdecor.h

    r29ed06d rdc0f6e4c  
    108108extern void ui_wdecor_close(ui_wdecor_t *);
    109109extern void ui_wdecor_move(ui_wdecor_t *, gfx_coord2_t *, sysarg_t);
    110 extern void ui_wdecor_resize(ui_wdecor_t *, ui_wdecor_rsztype_t,
     110extern void ui_wdecor_resize(ui_wdecor_t *, display_wnd_rsztype_t,
    111111    gfx_coord2_t *, sysarg_t);
    112112extern void ui_wdecor_set_cursor(ui_wdecor_t *, ui_stock_cursor_t);
    … …  
    116116extern errno_t ui_wdecor_sysmenu_hdl_paint(ui_wdecor_t *, gfx_rect_t *);
    117117extern void ui_wdecor_frame_pos_event(ui_wdecor_t *, pos_event_t *);
    118 extern ui_wdecor_rsztype_t ui_wdecor_get_rsztype(ui_wdecor_t *,
     118extern display_wnd_rsztype_t ui_wdecor_get_rsztype(ui_wdecor_t *,
    119119    gfx_coord2_t *);
    120 extern ui_stock_cursor_t ui_wdecor_cursor_from_rsztype(ui_wdecor_rsztype_t);
     120extern ui_stock_cursor_t ui_wdecor_cursor_from_rsztype(display_wnd_rsztype_t);
    121121
    122122#endif

  • uspace/lib/ui/src/wdecor.c

    r29ed06d rdc0f6e4c  
    650650 * @param pos_id Positioning device ID
    651651 */
    652 void ui_wdecor_resize(ui_wdecor_t *wdecor, ui_wdecor_rsztype_t rsztype,
     652void ui_wdecor_resize(ui_wdecor_t *wdecor, display_wnd_rsztype_t rsztype,
    653653    gfx_coord2_t *pos, sysarg_t pos_id)
    654654{
    … …  
    912912 * @return Resize type
    913913 */
    914 ui_wdecor_rsztype_t ui_wdecor_get_rsztype(ui_wdecor_t *wdecor,
     914display_wnd_rsztype_t ui_wdecor_get_rsztype(ui_wdecor_t *wdecor,
    915915    gfx_coord2_t *pos)
    916916{
    … …  
    923923        /* Window not resizable? */
    924924        if ((wdecor->style & ui_wds_resizable) == 0)
    925                 return ui_wr_none;
     925                return display_wr_none;
    926926
    927927        /* Window is maximized? */
    928928        if (wdecor->maximized)
    929                 return ui_wr_none;
     929                return display_wr_none;
    930930
    931931        /* Position not inside window? */
    932932        if (!gfx_pix_inside_rect(pos, &wdecor->rect))
    933                 return ui_wr_none;
     933                return display_wr_none;
    934934
    935935        /* Position is within edge width from the outside */
    … …  
    950950        /* Top-left corner */
    951951        if (edge && cleft && ctop)
    952                 return ui_wr_top_left;
     952                return display_wr_top_left;
    953953
    954954        /* Top-right corner */
    955955        if (edge && cright && ctop)
    956                 return ui_wr_top_right;
     956                return display_wr_top_right;
    957957
    958958        /* Bottom-left corner */
    959959        if (edge && cleft && cbottom)
    960                 return ui_wr_bottom_left;
     960                return display_wr_bottom_left;
    961961
    962962        /* Bottom-right corner */
    963963        if (edge && cright && cbottom)
    964                 return ui_wr_bottom_right;
     964                return display_wr_bottom_right;
    965965
    966966        /* Left edge */
    967967        if (eleft)
    968                 return ui_wr_left;
     968                return display_wr_left;
    969969
    970970        /* Right edge */
    971971        if (eright)
    972                 return ui_wr_right;
     972                return display_wr_right;
    973973
    974974        /* Top edge */
    975975        if (etop)
    976                 return ui_wr_top;
     976                return display_wr_top;
    977977
    978978        /* Bottom edge */
    979979        if (ebottom)
    980                 return ui_wr_bottom;
    981 
    982         return ui_wr_none;
     980                return display_wr_bottom;
     981
     982        return display_wr_none;
    983983}
    984984
    … …  
    990990 * @return Cursor to use for this resize type
    991991 */
    992 ui_stock_cursor_t ui_wdecor_cursor_from_rsztype(ui_wdecor_rsztype_t rsztype)
     992ui_stock_cursor_t ui_wdecor_cursor_from_rsztype(display_wnd_rsztype_t rsztype)
    993993{
    994994        switch (rsztype) {
    995         case ui_wr_none:
     995        case display_wr_none:
    996996                return ui_curs_arrow;
    997997
    998         case ui_wr_top:
    999         case ui_wr_bottom:
     998        case display_wr_top:
     999        case display_wr_bottom:
    10001000                return ui_curs_size_ud;
    10011001
    1002         case ui_wr_left:
    1003         case ui_wr_right:
     1002        case display_wr_left:
     1003        case display_wr_right:
    10041004                return ui_curs_size_lr;
    10051005
    1006         case ui_wr_top_left:
    1007         case ui_wr_bottom_right:
     1006        case display_wr_top_left:
     1007        case display_wr_bottom_right:
    10081008                return ui_curs_size_uldr;
    10091009
    1010         case ui_wr_top_right:
    1011         case ui_wr_bottom_left:
     1010        case display_wr_top_right:
     1011        case display_wr_bottom_left:
    10121012                return ui_curs_size_urdl;
    10131013
    … …  
    10741074        gfx_coord2_t pos;
    10751075        sysarg_t pos_id;
    1076         ui_wdecor_rsztype_t rsztype;
     1076        display_wnd_rsztype_t rsztype;
    10771077        ui_stock_cursor_t cursor;
    10781078
    … …  
    10891089
    10901090        /* Press on window border? */
    1091         if (rsztype != ui_wr_none && event->type == POS_PRESS)
     1091        if (rsztype != display_wr_none && event->type == POS_PRESS)
    10921092                ui_wdecor_resize(wdecor, rsztype, &pos, pos_id);
    10931093}

  • uspace/lib/ui/src/window.c

    r29ed06d rdc0f6e4c  
    8686static void wd_close(ui_wdecor_t *, void *);
    8787static void wd_move(ui_wdecor_t *, void *, gfx_coord2_t *, sysarg_t);
    88 static void wd_resize(ui_wdecor_t *, void *, ui_wdecor_rsztype_t,
     88static void wd_resize(ui_wdecor_t *, void *, display_wnd_rsztype_t,
    8989    gfx_coord2_t *, sysarg_t);
    9090static void wd_set_cursor(ui_wdecor_t *, void *, ui_stock_cursor_t);
    … …  
    11911191 */
    11921192static void wd_resize(ui_wdecor_t *wdecor, void *arg,
    1193     ui_wdecor_rsztype_t rsztype, gfx_coord2_t *pos, sysarg_t pos_id)
     1193    display_wnd_rsztype_t rsztype, gfx_coord2_t *pos, sysarg_t pos_id)
    11941194{
    11951195        ui_window_t *window = (ui_window_t *) arg;
    11961196
    11971197        if (window->dwindow != NULL) {
    1198                 (void) display_window_resize_req(window->dwindow,
    1199                     (display_wnd_rsztype_t) rsztype, // Same constants in the enums
     1198                (void) display_window_resize_req(window->dwindow, rsztype,
    12001199                    pos, pos_id);
    12011200        }

  • uspace/lib/ui/test/wdecor.c

    r29ed06d rdc0f6e4c  
    7272static void test_wdecor_close(ui_wdecor_t *, void *);
    7373static void test_wdecor_move(ui_wdecor_t *, void *, gfx_coord2_t *, sysarg_t);
    74 static void test_wdecor_resize(ui_wdecor_t *, void *, ui_wdecor_rsztype_t,
     74static void test_wdecor_resize(ui_wdecor_t *, void *, display_wnd_rsztype_t,
    7575    gfx_coord2_t *, sysarg_t);
    7676static void test_wdecor_set_cursor(ui_wdecor_t *, void *, ui_stock_cursor_t);
    … …  
    125125        char32_t accel;
    126126        bool resize;
    127         ui_wdecor_rsztype_t rsztype;
     127        display_wnd_rsztype_t rsztype;
    128128        bool set_cursor;
    129129        ui_stock_cursor_t cursor;
    … …  
    584584        ui_wdecor_t *wdecor;
    585585        test_cb_resp_t resp;
    586         ui_wdecor_rsztype_t rsztype;
     586        display_wnd_rsztype_t rsztype;
    587587        gfx_coord2_t pos;
    588588        sysarg_t pos_id;
    … …  
    591591        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
    592592
    593         rsztype = ui_wr_bottom;
     593        rsztype = display_wr_bottom;
    594594        pos.x = 3;
    595595        pos.y = 4;
    … …  
    605605        /* Resize callback with real callback set */
    606606        resp.resize = false;
    607         resp.rsztype = ui_wr_none;
     607        resp.rsztype = display_wr_none;
    608608        resp.pos.x = 0;
    609609        resp.pos.y = 0;
    … …  
    13611361        ui_wdecor_t *wdecor;
    13621362        gfx_rect_t rect;
    1363         ui_wdecor_rsztype_t rsztype;
     1363        display_wnd_rsztype_t rsztype;
    13641364        gfx_coord2_t pos;
    13651365        errno_t rc;
    … …  
    13871387        pos.y = -1;
    13881388        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1389         PCUT_ASSERT_EQUALS(ui_wr_none, rsztype);
     1389        PCUT_ASSERT_EQUALS(display_wr_none, rsztype);
    13901390
    13911391        /* Middle of the window */
    … …  
    13931393        pos.y = 100;
    13941394        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1395         PCUT_ASSERT_EQUALS(ui_wr_none, rsztype);
     1395        PCUT_ASSERT_EQUALS(display_wr_none, rsztype);
    13961396
    13971397        /* Top-left corner, but not on edge */
    … …  
    13991399        pos.y = 30;
    14001400        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1401         PCUT_ASSERT_EQUALS(ui_wr_none, rsztype);
     1401        PCUT_ASSERT_EQUALS(display_wr_none, rsztype);
    14021402
    14031403        /* Top-left corner on top edge */
    … …  
    14051405        pos.y = 20;
    14061406        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1407         PCUT_ASSERT_EQUALS(ui_wr_top_left, rsztype);
     1407        PCUT_ASSERT_EQUALS(display_wr_top_left, rsztype);
    14081408
    14091409        /* Top-left corner on left edge */
    … …  
    14111411        pos.y = 30;
    14121412        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1413         PCUT_ASSERT_EQUALS(ui_wr_top_left, rsztype);
     1413        PCUT_ASSERT_EQUALS(display_wr_top_left, rsztype);
    14141414
    14151415        /* Top-right corner on top edge */
    … …  
    14171417        pos.y = 20;
    14181418        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1419         PCUT_ASSERT_EQUALS(ui_wr_top_right, rsztype);
     1419        PCUT_ASSERT_EQUALS(display_wr_top_right, rsztype);
    14201420
    14211421        /* Top-right corner on right edge */
    … …  
    14231423        pos.y = 30;
    14241424        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1425         PCUT_ASSERT_EQUALS(ui_wr_top_right, rsztype);
     1425        PCUT_ASSERT_EQUALS(display_wr_top_right, rsztype);
    14261426
    14271427        /* Top edge */
    … …  
    14291429        pos.y = 20;
    14301430        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1431         PCUT_ASSERT_EQUALS(ui_wr_top, rsztype);
     1431        PCUT_ASSERT_EQUALS(display_wr_top, rsztype);
    14321432
    14331433        /* Bottom edge */
    … …  
    14351435        pos.y = 199;
    14361436        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1437         PCUT_ASSERT_EQUALS(ui_wr_bottom, rsztype);
     1437        PCUT_ASSERT_EQUALS(display_wr_bottom, rsztype);
    14381438
    14391439        /* Left edge */
    … …  
    14411441        pos.y = 100;
    14421442        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1443         PCUT_ASSERT_EQUALS(ui_wr_left, rsztype);
     1443        PCUT_ASSERT_EQUALS(display_wr_left, rsztype);
    14441444
    14451445        /* Right edge */
    … …  
    14471447        pos.y = 100;
    14481448        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1449         PCUT_ASSERT_EQUALS(ui_wr_right, rsztype);
     1449        PCUT_ASSERT_EQUALS(display_wr_right, rsztype);
    14501450
    14511451        ui_wdecor_destroy(wdecor);
    … …  
    14661466        pos.y = 20;
    14671467        rsztype = ui_wdecor_get_rsztype(wdecor, &pos);
    1468         PCUT_ASSERT_EQUALS(ui_wr_none, rsztype);
     1468        PCUT_ASSERT_EQUALS(display_wr_none, rsztype);
    14691469
    14701470        ui_wdecor_destroy(wdecor);
    … …  
    14791479{
    14801480        PCUT_ASSERT_EQUALS(ui_curs_arrow,
    1481             ui_wdecor_cursor_from_rsztype(ui_wr_none));
     1481            ui_wdecor_cursor_from_rsztype(display_wr_none));
    14821482        PCUT_ASSERT_EQUALS(ui_curs_size_ud,
    1483             ui_wdecor_cursor_from_rsztype(ui_wr_top));
     1483            ui_wdecor_cursor_from_rsztype(display_wr_top));
    14841484        PCUT_ASSERT_EQUALS(ui_curs_size_ud,
    1485             ui_wdecor_cursor_from_rsztype(ui_wr_bottom));
     1485            ui_wdecor_cursor_from_rsztype(display_wr_bottom));
    14861486        PCUT_ASSERT_EQUALS(ui_curs_size_lr,
    1487             ui_wdecor_cursor_from_rsztype(ui_wr_left));
     1487            ui_wdecor_cursor_from_rsztype(display_wr_left));
    14881488        PCUT_ASSERT_EQUALS(ui_curs_size_lr,
    1489             ui_wdecor_cursor_from_rsztype(ui_wr_right));
     1489            ui_wdecor_cursor_from_rsztype(display_wr_right));
    14901490        PCUT_ASSERT_EQUALS(ui_curs_size_uldr,
    1491             ui_wdecor_cursor_from_rsztype(ui_wr_top_left));
     1491            ui_wdecor_cursor_from_rsztype(display_wr_top_left));
    14921492        PCUT_ASSERT_EQUALS(ui_curs_size_uldr,
    1493             ui_wdecor_cursor_from_rsztype(ui_wr_bottom_right));
     1493            ui_wdecor_cursor_from_rsztype(display_wr_bottom_right));
    14941494        PCUT_ASSERT_EQUALS(ui_curs_size_urdl,
    1495             ui_wdecor_cursor_from_rsztype(ui_wr_top_right));
     1495            ui_wdecor_cursor_from_rsztype(display_wr_top_right));
    14961496        PCUT_ASSERT_EQUALS(ui_curs_size_urdl,
    1497             ui_wdecor_cursor_from_rsztype(ui_wr_bottom_left));
     1497            ui_wdecor_cursor_from_rsztype(display_wr_bottom_left));
    14981498}
    14991499
    … …  
    17251725
    17261726static void test_wdecor_resize(ui_wdecor_t *wdecor, void *arg,
    1727     ui_wdecor_rsztype_t rsztype, gfx_coord2_t *pos, sysarg_t pos_id)
     1727    display_wnd_rsztype_t rsztype, gfx_coord2_t *pos, sysarg_t pos_id)
    17281728{
    17291729        test_cb_resp_t *resp = (test_cb_resp_t *) arg;
Note: See TracChangeset for help on using the changeset viewer.