Re: [longo] wmaker -- alertas do compilador
Herbert P Fortes Neto <[EMAIL PROTECTED]> writes: > >> Penso que voce deveria dividir o patch em dois. Correcoes de problemas >> em 1 e remocao de warnings e similares em outro. Depois reportar 2 >> tickets no pacote para que o mantenedor possa entao aplica-los ou >> envia-los para o upstream. > > Obrigado pela resposta. > > Mais um esclarecimento. Correções de problemas seria para o retorno > da função W_InitIMStuff e para a definição 'static void' em > handleXkbIndicatorStateNotify ? Sim. Eh interessante dividir os patches por unidades logicas. Melhorias gerais no codigo e reais problemas pois fica mais facil para o mantenedor lidar com os patches alem do upstream decidir o que ele vai ou nao aplicar. Outra coisa importante eh voce descriminar o que cada patch faz e por que isso eh necessario ou benefico para o codigo ou projeto. -- O T A V I OS A L V A D O R - E-mail: [EMAIL PROTECTED] UIN: 5906116 GNU/Linux User: 239058 GPG ID: 49A5F855 Home Page: http://www.freedom.ind.br/otavio - "Microsoft gives you Windows ... Linux gives you the whole house."
Re: [longo] wmaker -- alertas do compilador
> Penso que voce deveria dividir o patch em dois. Correcoes de problemas > em 1 e remocao de warnings e similares em outro. Depois reportar 2 > tickets no pacote para que o mantenedor possa entao aplica-los ou > envia-los para o upstream. Obrigado pela resposta. Mais um esclarecimento. Correções de problemas seria para o retorno da função W_InitIMStuff e para a definição 'static void' em handleXkbIndicatorStateNotify ? [] hpfn -- Linux user number 416100 Ging FE1905B2 - ID pgpffR30HgTTW.pgp Description: PGP signature
Re: [longo] wmaker -- alertas do compilador
Herbert P Fortes Neto <[EMAIL PROTECTED]> writes: > Algumas alterações seriam aplicáveis na versão etch. Gostaria de saber > se concordam com as alterações, se deveria reportar bug(sarge,etch), > sobre o retorno da função logo acima. Penso que voce deveria dividir o patch em dois. Correcoes de problemas em 1 e remocao de warnings e similares em outro. Depois reportar 2 tickets no pacote para que o mantenedor possa entao aplica-los ou envia-los para o upstream. -- O T A V I OS A L V A D O R - E-mail: [EMAIL PROTECTED] UIN: 5906116 GNU/Linux User: 239058 GPG ID: 49A5F855 Home Page: http://www.freedom.ind.br/otavio - "Microsoft gives you Windows ... Linux gives you the whole house."
[longo] wmaker -- alertas do compilador
Olá Retirei alguns alertas do compilador reconstruíndo o pacote wmaker(stable). Gostaria de ouvir opiniões. -- 'unused variable' (apaguei) -wmaker-0.91.0/WINGs/configuration.c:56: warning: unused variable `str' -wmaker-0.91.0/src/window.c:3307: warning: unused variable `staterec' -wmaker-0.91.0/util/fontconv.c:133: warning: unused variable `slt' -- 'defined but not used' (comentei) -wmaker-0.91.0/WPrefs.app/Themes.c:151: warning: `updateThemeList' defined but not used -- 'braces' -wmaker-0.91.0/src/framewin.c:156: warning: suggest explicit braces to avoid ambiguous `else' -- 'implicit declaration' -coloquei um '#include ' para wmaker-0.91.0/src/startup.c:799: warning: implicit declaration of function `XkbQueryExtension' -coloquei o protótipo, acrescentando 'static void' para wmaker-0.91.0/src/event.c:496: warning: implicit declaration of function `handleXkbIndicatorStateNotify' /tmp/buildd/wmaker-0.91.0/src/event.c:1204: warning: return type defaults to `int' /tmp/buildd/wmaker-0.91.0/src/event.c:1226: warning: control reaches end of non-void function a função: #ifdef KEEP_XKB_LOCK_STATUS /* please help ]d if you know what to do */ handleXkbIndicatorStateNotify(XEvent *event) { WWindow *wwin; WScreen *scr; XkbStateRec staterec; int i; for (i=0; ifocused_window; if (wwin && wwin->flags.focused) { XkbGetState(dpy,XkbUseCoreKbd,&staterec); if (wwin->frame->languagemode != staterec.group) { wwin->frame->last_languagemode = wwin->frame->languagemode; wwin->frame->languagemode = staterec.group; } #ifdef XKB_BUTTON_HINT if (wwin->frame->titlebar) { wFrameWindowPaint(wwin->frame); } #endif } } } #endif /*KEEP_XKB_LOCK_STATUS*/ - coloquei o protótipo em wmaker-0.91.0.hpfn/WINGs/WINGs/WINGsP.h (parece ser como está na versão do etch -- com nome abreviado) para wmaker-0.91.0/WINGs/widgets.c:635: warning: implicit declaration of function `W_InitIMStuff' Na função eu diria estar faltando um return True, mas não mexi. A função: Bool W_InitIMStuff(WMScreen *scr) { WMIMContext *ctx; ctx = scr->imctx = wmalloc(sizeof(WMIMContext)); ctx->xim = XOpenIM(scr->display, NULL, NULL, NULL); if (ctx->xim == NULL) { wwarning("could not open IM"); return False; } } Algumas alterações seriam aplicáveis na versão etch. Gostaria de saber se concordam com as alterações, se deveria reportar bug(sarge,etch), sobre o retorno da função logo acima. [] hpfn -- Linux user number 416100 Ging FE1905B2 - ID pgpZpqYCNKayv.pgp Description: PGP signature