Changeset 7b4e7dc93dd9bf941bcfe2e2911bbf1de86d7bc6

Show
Ignore:
Timestamp:
07/05/08 21:33:55 (5 months ago)
Author:
Felix Paul Kühne <fkuehne@videolan.org>
git-committer:
Felix Paul Kühne <fkuehne@videolan.org> 1210188835 +0200
git-parent:

[82921f3c8dae23178b64089b22502bb8f528e7e6]

git-author:
Felix Paul Kühne <fkuehne@videolan.org> 1210188813 +0200
Message:

Remove any pl_Get usage

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • modules/gui/ncurses.c

    r2b5eb31 r7b4e7dc  
    109109 
    110110static void start_color_and_pairs ( intf_thread_t * ); 
    111 static playlist_t *pl_Get( intf_thread_t *p_intf ) 
    112 { 
    113     return p_intf->p_sys->p_playlist; 
    114 } 
    115111 
    116112/***************************************************************************** 
    …… 
    570566    intf_sys_t *p_sys = p_intf->p_sys; 
    571567    vlc_value_t val; 
    572     playlist_t *p_playlist = pl_Get( p_intf ); 
     568     
     569    #define ReturnTrue \ 
     570    vlc_object_release( p_playlist ); \ 
     571    return 1 
     572     
     573    #define ReturnFalse \ 
     574    vlc_object_release( p_playlist ); \ 
     575    return 0 
     576 
     577    playlist_t *p_playlist = pl_Yield( p_intf ); 
    573578 
    574579    if( p_sys->i_box_type == BOX_PLAYLIST ) 
    …… 
    585590                val.b_bool = !val.b_bool; 
    586591                var_Set( p_playlist, "random", val ); 
    587                 return 1; 
     592                vlc_object_release( p_playlist ); 
     593                ReturnTrue; 
    588594            case 'l': 
    589595                var_Get( p_playlist, "loop", &val ); 
    590596                val.b_bool = !val.b_bool; 
    591597                var_Set( p_playlist, "loop", val ); 
    592                 return 1; 
     598                vlc_object_release( p_playlist ); 
     599                ReturnTrue; 
    593600            case 'R': 
    594601                var_Get( p_playlist, "repeat", &val ); 
    595602                val.b_bool = !val.b_bool; 
    596603                var_Set( p_playlist, "repeat", val ); 
    597                 return 1; 
     604                vlc_object_release( p_playlist ); 
     605                ReturnTrue; 
    598606 
    599607            /* Playlist sort */ 
    …… 
    603611                                            SORT_TITLE_NODES_FIRST, ORDER_NORMAL ); 
    604612                p_sys->b_need_update = true; 
    605                 return 1; 
     613                ReturnTrue; 
    606614            case 'O': 
    607615                playlist_RecursiveNodeSort( p_playlist, 
    …… 
    609617                                            SORT_TITLE_NODES_FIRST, ORDER_REVERSE ); 
    610618                p_sys->b_need_update = true; 
    611                 return 1; 
     619                ReturnTrue; 
    612620 
    613621            /* Playlist view */ 
    …… 
    623631                //p_sys->b_need_update = true; 
    624632                PlaylistRebuild( p_intf ); 
    625                 return 1; 
     633                ReturnTrue; 
    626634 
    627635            /* Playlist navigation */ 
    …… 
    726734                b_box_plidx_follow = true; 
    727735            p_sys->b_box_plidx_follow = b_box_plidx_follow; 
    728             return 1; 
     736            ReturnTrue; 
    729737        } 
    730738    } 
    …… 
    814822            if( p_sys->i_box_bidx >= p_sys->i_dir_entries ) p_sys->i_box_bidx = p_sys->i_dir_entries - 1; 
    815823            if( p_sys->i_box_bidx < 0 ) p_sys->i_box_bidx = 0; 
    816             return 1; 
     824            ReturnTrue; 
    817825        } 
    818826    } 
    …… 
    825833            case KEY_HOME: 
    826834                p_sys->i_box_start = 0; 
    827                 return 1; 
     835                ReturnTrue; 
    828836#ifdef __FreeBSD__ 
    829837            case KEY_SELECT: 
    …… 
    831839            case KEY_END: 
    832840                p_sys->i_box_start = p_sys->i_box_lines_total - 1; 
    833                 return 1; 
     841                ReturnTrue; 
    834842            case KEY_UP: 
    835843                if( p_sys->i_box_start > 0 ) p_sys->i_box_start--; 
    836                 return 1; 
     844                ReturnTrue; 
    837845            case KEY_DOWN: 
    838846                if( p_sys->i_box_start < p_sys->i_box_lines_total - 1 ) 
    …… 
    840848                    p_sys->i_box_start++; 
    841849                } 
    842                 return 1; 
     850                ReturnTrue; 
    843851            case KEY_PPAGE: 
    844852                p_sys->i_box_start -= p_sys->i_box_lines; 
    845853                if( p_sys->i_box_start < 0 ) p_sys->i_box_start = 0; 
    846                 return 1; 
     854                ReturnTrue; 
    847855            case KEY_NPAGE: 
    848856                p_sys->i_box_start += p_sys->i_box_lines; 
    …… 
    851859                    p_sys->i_box_start = p_sys->i_box_lines_total - 1; 
    852860                } 
    853                 return 1; 
     861                ReturnTrue; 
    854862            default: 
    855863                break; 
    …… 
    863871                p_sys->f_slider = 0; 
    864872                ManageSlider( p_intf ); 
    865                 return 1; 
     873                ReturnTrue; 
    866874#ifdef __FreeBSD__ 
    867875            case KEY_SELECT: 
    …… 
    870878                p_sys->f_slider = 99.9; 
    871879                ManageSlider( p_intf ); 
    872                 return 1; 
     880                ReturnTrue; 
    873881            case KEY_UP: 
    874882                p_sys->f_slider += 5.0; 
    875883                if( p_sys->f_slider >= 99.0 ) p_sys->f_slider = 99.0; 
    876884                ManageSlider( p_intf ); 
    877                 return 1; 
     885                ReturnTrue; 
    878886            case KEY_DOWN: 
    879887                p_sys->f_slider -= 5.0; 
    880888                if( p_sys->f_slider < 0.0 ) p_sys->f_slider = 0.0; 
    881889                ManageSlider( p_intf ); 
    882                 return 1; 
     890                ReturnTrue; 
    883891 
    884892            default: 
    …… 
    894902            case 0x0c:      /* ^l */ 
    895903                clear(); 
    896                 return 1; 
     904                ReturnTrue; 
    897905            case KEY_ENTER: 
    898906            case '\r': 
    …… 
    907915                } 
    908916                p_sys->i_box_type = BOX_PLAYLIST; 
    909                 return 1; 
     917                ReturnTrue; 
    910918            case 0x1b: /* ESC */ 
    911919                /* Alt+key combinations return 2 keys in the terminal keyboard: 
    …… 
    924932                 */ 
    925933                if( wgetch( p_sys->w ) != ERR ) 
    926                     return 0; 
     934                    ReturnFalse; 
    927935                p_sys->i_box_plidx = p_sys->i_before_search; 
    928936                p_sys->i_box_type = BOX_PLAYLIST; 
    929                 return 1; 
     937                ReturnTrue; 
    930938            case KEY_BACKSPACE: 
    931939            case 0x7f: 
    …… 
    959967        p_sys->psz_old_search = NULL; 
    960968        SearchPlaylist( p_intf, p_sys->psz_search_chain ); 
    961         return 1; 
     969        ReturnTrue; 
    962970    } 
    963971    else if( p_sys->i_box_type == BOX_OPEN && p_sys->psz_open_chain ) 
    …… 
    970978            case 0x0c:          /* ^l */ 
    971979                clear(); 
    972                 return 1; 
     980                ReturnTrue; 
    973981            case KEY_ENTER: 
    974982            case '\r':  
    …… 
    9951003                } 
    9961004                p_sys->i_box_type = BOX_PLAYLIST; 
    997                 return 1; 
     1005                ReturnTrue; 
    9981006            case 0x1b:  /* ESC */ 
    9991007                if( wgetch( p_sys->w ) != ERR ) 
    1000                     return 0; 
     1008                    ReturnFalse; 
    10011009                p_sys->i_box_type = BOX_PLAYLIST; 
    1002                 return 1; 
     1010                ReturnTrue; 
    10031011            case KEY_BACKSPACE: 
    10041012            case 0x7f: 
    …… 
    10291037            } 
    10301038        } 
    1031         return 1; 
     1039        ReturnTrue; 
    10321040    } 
    10331041 
    …… 
    10381046        case 0x1b:  /* ESC */ 
    10391047            if( wgetch( p_sys->w ) != ERR ) 
    1040                 return 0; 
     1048                ReturnFalse; 
    10411049        case 'q': 
    10421050        case 'Q': 
    10431051        case KEY_EXIT: 
    10441052            vlc_object_kill( p_intf->p_libvlc ); 
    1045             return 0; 
     1053            ReturnFalse; 
    10461054 
    10471055        /* Box switching */ 
    …… 
    10521060                p_sys->i_box_type = BOX_INFO; 
    10531061            p_sys->i_box_lines_total = 0; 
    1054             return 1; 
     1062            ReturnTrue; 
    10551063        case 'm': 
    10561064            if( p_sys->i_box_type == BOX_META ) 
    …… 
    10591067                p_sys->i_box_type = BOX_META; 
    10601068            p_sys->i_box_lines_total = 0; 
    1061             return 1; 
     1069            ReturnTrue; 
    10621070        case 'L': 
    10631071            if( p_sys->i_box_type == BOX_LOG ) 
    …… 
    10651073            else 
    10661074                p_sys->i_box_type = BOX_LOG; 
    1067             return 1; 
     1075            ReturnTrue; 
    10681076        case 'P': 
    10691077            if( p_sys->i_box_type == BOX_PLAYLIST ) 
    …… 
    10711079            else 
    10721080                p_sys->i_box_type = BOX_PLAYLIST; 
    1073             return 1; 
     1081            ReturnTrue; 
    10741082        case 'B': 
    10751083            if( p_sys->i_box_type == BOX_BROWSE ) 
    …… 
    10771085            else 
    10781086                p_sys->i_box_type = BOX_BROWSE; 
    1079             return 1; 
     1087            ReturnTrue; 
    10801088        case 'x': 
    10811089            if( p_sys->i_box_type == BOX_OBJECTS ) 
    …… 
    10831091            else 
    10841092                p_sys->i_box_type = BOX_OBJECTS; 
    1085             return 1; 
     1093            ReturnTrue; 
    10861094        case 'S': 
    10871095            if( p_sys->i_box_type == BOX_STATS ) 
    …… 
    10891097            else 
    10901098                p_sys->i_box_type = BOX_STATS; 
    1091             return 1; 
     1099            ReturnTrue; 
    10921100        case 'c': 
    10931101            p_sys->b_color = !p_sys->b_color; 
    10941102            if( p_sys->b_color && !p_sys->b_color_started ) 
    10951103                start_color_and_pairs( p_intf ); 
    1096             return 1; 
     1104            ReturnTrue; 
    10971105        case 'h': 
    10981106        case 'H': 
    …… 
    11021110                p_sys->i_box_type = BOX_HELP; 
    11031111            p_sys->i_box_lines_total = 0; 
    1104             return 1; 
     1112            ReturnTrue; 
    11051113        case '/': 
    11061114            if( p_sys->i_box_type != BOX_SEARCH ) 
    11071115            { 
    11081116                if( p_sys->psz_search_chain == NULL ) 
    1109                 { 
    1110                     return 1; 
    1111                 } 
     1117                    ReturnTrue; 
    11121118                p_sys->psz_search_chain[0] = '\0'; 
    11131119                p_sys->b_box_plidx_follow = false; 
    …… 
    11151121                p_sys->i_box_type = BOX_SEARCH; 
    11161122            } 
    1117             return 1; 
     1123            ReturnTrue; 
    11181124        case 'A': /* Open */ 
    11191125            if( p_sys->i_box_type != BOX_OPEN ) 
    11201126            { 
    11211127                if( p_sys->psz_open_chain == NULL ) 
    1122                 { 
    1123                     return 1; 
    1124                 } 
     1128                    ReturnTrue; 
    11251129                p_sys->psz_open_chain[0] = '\0'; 
    11261130                p_sys->i_box_type = BOX_OPEN; 
    11271131            } 
    1128             return 1; 
     1132            ReturnTrue; 
    11291133 
    11301134        /* Navigation */ 
    …… 
    11331137            if( p_sys->f_slider > 99.9 ) p_sys->f_slider = 99.9; 
    11341138            ManageSlider( p_intf ); 
    1135             return 1; 
     1139            ReturnTrue; 
    11361140 
    11371141        case KEY_LEFT: 
    …… 
    11391143            if( p_sys->f_slider < 0.0 ) p_sys->f_slider = 0.0; 
    11401144            ManageSlider( p_intf ); 
    1141             return 1; 
     1145            ReturnTrue; 
    11421146 
    11431147        /* Common control */ 
    …… 
    11631167                } 
    11641168            } 
    1165             return 0; 
     1169            ReturnFalse; 
    11661170        } 
    11671171 
    11681172        case ' ': 
    11691173            PlayPause( p_intf ); 
    1170             return 1; 
     1174            ReturnTrue; 
    11711175 
    11721176        case 's': 
    11731177            playlist_Stop( p_playlist ); 
    1174             return 1; 
     1178            ReturnTrue; 
    11751179 
    11761180        case 'e': 
    11771181            Eject( p_intf ); 
    1178             return 1; 
     1182            ReturnTrue; 
    11791183 
    11801184        case '[': 
    …… 
    11841188                var_Set( p_sys->p_input, "prev-title", val ); 
    11851189            } 
    1186             return 1; 
     1190            ReturnTrue; 
    11871191 
    11881192        case ']': 
    …… 
    11921196                var_Set( p_sys->p_input, "next-title", val ); 
    11931197            } 
    1194             return 1; 
     1198            ReturnTrue; 
    11951199 
    11961200        case '<': 
    …… 
    12001204                var_Set( p_sys->p_input, "prev-chapter", val ); 
    12011205            } 
    1202             return 1; 
     1206            ReturnTrue; 
    12031207 
    12041208        case '>': 
    …… 
    12081212                var_Set( p_sys->p_input, "next-chapter", val ); 
    12091213            } 
    1210             return 1; 
     1214            ReturnTrue; 
    12111215 
    12121216        case 'p': 
    12131217            playlist_Prev( p_playlist ); 
    12141218            clear(); 
    1215             return 1; 
     1219            ReturnTrue; 
    12161220 
    12171221        case 'n': 
    12181222            playlist_Next( p_playlist ); 
    12191223            clear(); 
    1220             return 1; 
     1224            ReturnTrue; 
    12211225 
    12221226        case 'a': 
    12231227            aout_VolumeUp( p_intf, 1, NULL ); 
    12241228            clear(); 
    1225             return 1; 
     1229            ReturnTrue; 
    12261230 
    12271231        case 'z': 
    12281232            aout_VolumeDown( p_intf, 1, NULL ); 
    12291233            clear(); 
    1230             return 1; 
     1234            ReturnTrue; 
    12311235 
    12321236        /* 
    …… 
    12361240        case 0x0c:          /* ^l */ 
    12371241            clear(); 
    1238             return 1; 
     1242            ReturnTrue; 
    12391243 
    12401244        default: 
    1241             return 0; 
     1245            ReturnFalse; 
    12421246    } 
    12431247} 
    …… 
    15001504    intf_sys_t     *p_sys = p_intf->p_sys; 
    15011505    input_thread_t *p_input = p_sys->p_input; 
    1502     playlist_t     *p_playlist = pl_Get( p_intf ); 
     1506    playlist_t     *p_playlist = pl_Yield( p_intf ); 
    15031507    int y = 0; 
    15041508    int h; 
    …… 
    22322236 
    22332237    *t_last_refresh = time( 0 ); 
     2238    vlc_object_release( p_playlist ); 
    22342239} 
    22352240 
    …… 
    22372242{ 
    22382243    intf_sys_t *p_sys = p_intf->p_sys; 
    2239     playlist_t *p_playlist = pl_Get( p_intf ); 
     2244    playlist_t *p_playlist = pl_Yield( p_intf ); 
     2245    playlist_item_t *p_item; 
    22402246 
    22412247    switch( p_sys->i_current_view ) 
    22422248    { 
    22432249        case VIEW_CATEGORY: 
    2244             return p_playlist->p_root_category; 
     2250            p_item = p_playlist->p_root_category; 
    22452251        default: 
    2246             return p_playlist->p_root_onelevel; 
    2247     } 
     2252            p_item = p_playlist->p_root_onelevel; 
     2253    } 
     2254    vlc_object_release( p_playlist ); 
     2255    return p_item; 
    22482256} 
    22492257 
    …… 
    22512259{ 
    22522260    intf_sys_t *p_sys = p_intf->p_sys; 
    2253     playlist_t *p_playlist = pl_Get( p_intf ); 
     2261    playlist_t *p_playlist = pl_Yield( p_intf ); 
    22542262 
    22552263    PL_LOCK; 
    …… 
    22642272 
    22652273    PL_UNLOCK; 
     2274     
     2275    vlc_object_release( p_playlist ); 
    22662276} 
    22672277 
    …… 
    23242334    VLC_UNUSED(oval); VLC_UNUSED(nval); 
    23252335    intf_thread_t *p_intf = (intf_thread_t *)param; 
    2326     playlist_t *p_playlist = pl_Get( p_intf ); 
     2336    playlist_t *p_playlist = pl_Yield( p_intf ); 
    23272337    p_intf->p_sys->b_need_update = true; 
    23282338    p_intf->p_sys->p_node = p_playlist->status.p_node; 
     2339    vlc_object_release( p_playlist ); 
    23292340    return VLC_SUCCESS; 
    23302341} 
    …… 
    23342345                                            playlist_item_t *p_item ) 
    23352346{ 
    2336     playlist_t *p_playlist = pl_Get( p_intf ); 
     2347    playlist_t *p_playlist = pl_Yield( p_intf ); 
    23372348    playlist_item_t *p_played_item = p_playlist->status.p_item; 
     2349    vlc_object_release( p_playlist ); 
    23382350    return( p_item != NULL && p_played_item != NULL && 
    23392351            p_item->p_input != NULL && p_played_item->p_input != NULL && 
    …… 
    23922404     */ 
    23932405 
    2394     playlist_t * p_playlist = pl_Get( p_intf ); 
     2406    playlist_t * p_playlist = pl_Yield( p_intf ); 
    23952407    PL_LOCK; 
    23962408 
    …… 
    23982410    { 
    23992411        PL_UNLOCK; 
     2412        vlc_object_release( p_playlist ); 
    24002413        return; 
    24012414    } 
    …… 
    24802493 
    24812494    free( psz_device ); 
     2495    vlc_object_release( p_playlist ); 
    24822496    return; 
    24832497} 
    …… 
    26012615{ 
    26022616    input_thread_t *p_input = p_intf->p_sys->p_input; 
    2603     playlist_t *p_playlist = pl_Get( p_intf ); 
     2617    playlist_t *p_playlist = pl_Yield( p_intf ); 
    26042618    vlc_value_t val; 
    26052619 
    …… 
    26192633    else 
    26202634        playlist_Play( p_playlist ); 
     2635 
     2636    vlc_object_release( p_playlist ); 
    26212637} 
    26222638