Opened 14 years ago

Closed 14 years ago

#3452 closed defect (fixed)

windows issues with error functions

Reported by: assefa Owned by: mapserverbugs
Priority: normal Milestone: 6.0 release
Component: MapScript-PHP Version: unspecified
Severity: normal Keywords:
Cc:

Description

error functions such as mapscript_report_php_error give a problem on windows (which is build in thread safe mode).

For the record here is part of the conversation with Alan:

Magie..... ou Intelligence des software Unix! huhu :)

Alan

On May 7, 2010 09:33:35 am Yewondwossen Assefa wrote:

ah, c'est de la magie GCC :) Je vais faire les commits tantot.

merci.

On 07/05/2010 9:27 AM, Alan Boudreault wrote:

Bon, voilà, j'ai trouvé. GCC n'utilise tout simplement pas le second argument et est assé intelligent pour trouver les autres arguments passés par lui même:

http://www.delorie.com/gnu/docs/gcc/gccint_135.html << The user is supposed to write the last named argument of the function here. However, va_start should not use this argument. The way to find the end of the named arguments is with the built-in functions described below. >>

Voilà pourquoi cela fonctionnait chez moi. Afin de régler ce problème sur tous les systèmes, ton fix serait approprié. Tu peux donc committer quand tu es prêts. Il faudrait faire la même chose pour la fonction mapscript_throw_exception et mapscript_throw_mapserver_exception. Si tu préfrère, je peux faire les corrections.

Merci

Alan

On May 7, 2010 09:05:06 am Yewondwossen Assefa wrote:

Ce n'est pas un problème avec le call de la fonction a ce que je vois,

c'est un problème avec la fonction va_start utilisée dans la fonction;

elle est sensée je crois prendre tous ales arguments venant apres la

2ieme argument, donc

va_start(args, format); va prendre tous les arguments apres format,

donc thread_id est compris lorsque ce defini.

Ca fait-tu du sens? La modification que j'ai fait est par rapport a ca

et cela fonctionne pour moi sur windows. Est-ce que cela fonctionne pour

toi?

Change History (1)

comment:1 by assefa, 14 years ago

Resolution: fixed
Status: newclosed

committed in r10143

Note: See TracTickets for help on using tickets.