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?
committed in r10143