Re: [longo] wmaker -- alertas do compilador
On Thu, 07 Sep 2006 19:46:24 -0300 Otavio Salvador <[EMAIL PROTECTED]> wrote: > 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. Tá bom, obrigado pela ajuda. [] hpfn -- Linux user number 416100 Ging FE1905B2 - ID pgpfBKk0k05V0.pgp Description: PGP signature
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
