Alas for me. I am one of those who found python hazy and ruby clear, so I turned down the ruby path and know nothing of python.

Roger

http://thread.gmane.org/gmane.comp.python.brasil/12593

Bom, manda o patch renovado para lá que eu vejo o que ficou “faltando” :-)

Já mudei o patch para usar ‘else if’ e tirei os continues. Pra você ver como fazer um patch ‘nas coxas’ só aumenta o trabalho :)

Não entendi o que você quis dizer no parágrafo que fala sobre espaços em branco e sobre os testes duplicados (criei uma thread lá na lista, vamos mudar a discussão pra lá?)

strcpy(): Eu sei que é harmful, mas mantive a mesma construção do original. Sim, o import.c original usa strcpy() e a verificação de “strlen()

Sobre o Patch, legal, mas alguns comentários que podem te ajudar a tê-lo aceito:

- if (strcmp(filetab->suffix, “.pyc”) == 0)
+ if (strcmp(filetab->suffix, “.pyc”) == 0) {
filetab->suffix = “.pyo”;
+ continue;
+ }
+ if (strcmp(filetab->suffix, “.pyo”) == 0) {
+ filetab->suffix = “.pyc”;
+ continue;
+ }

pode ser substituído por:

+ else if (strcmp(filetab->suffix, “.pyo”) == 0)
+ filetab->suffix = “.pyc”;

eliminando os “continue“.

Depois, tem algo não relevante, em “/* Try .py */” juntamente com a linha em branco acima. Idem para “/* Try pyo/pyc */” e as mudanças em whitespace no bloco todo! Aliás, a repetição deste bloco deveria ser feita dentro do primeiro, para os valores diferentes “(!Py_OptimizeFlag)“, você está conferindo a mesma coisa duas vezes! :-).

Tirar o “/* XXX INDT */” ajuda :-)

strcpy() is harmful! Troque por:

strncpy(module_filename, filename, MAXPATHLEN);
module_filename[ MAXPATHLEN ] = ”; /* vide man strncpy por que */

com isso a comparação “strlen(module_filename) ” torna-se sempre verdadeira (no seu código atual se ela for falsa –> SEGFAULT!)

Então você deve trocar a comparação por MAXPATHLEN-1, para caber o “c” ou “o” da extensão.

Uma pequena optimização nesta última parte é fazer:

size_t l = strlen( module_filename );

Valeu!

Eu criei um patch de sugestão em
http://pymaemo.sourceforge.net/cgi-bin/moin.cgi/OsvaldoSantanaNeto

Roger,

Thanks for the quote in your blog. I’ll try to publish my posts in english in near future to make reader’s life easier :)

Eu concordo que o comportamento do zipimport eh o mais adequado.

Eu ja tropecei no mesmo problema mas nao tinha notado que o zipimport se comportava de forma diferente. Seria otimo se isso fosse consertado.