[spirit] Re: Boost regression notification (2006-02-24 [HEAD])

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[spirit] Re: Boost regression notification (2006-02-24 [HEAD])

Bugzilla from markus.schoepflin@comsoft.de
Douglas Gregor wrote:

> |spirit|
>   scanner_value_type_tests:  cw-8_3  cw-9_4  cw-9_5-darwin  gcc-3.2.3-linux  gcc-3.3.6-linux  gcc-3.3.6-linux  gcc-3.4.4-linux  gcc-3.4.5-linux  gcc-3_3-darwin  gcc-4.0.2-linux  gcc-4_0-darwin  intel-9.0-linux  intel-win32-8_1  intel-win32-9_0  qcc-3.3.5-cpp  tru64cxx71-006  vc-7_1  vc-7_1  vc-7_1
>   scanner_value_type_tests_debug:  cw-8_3  cw-9_4  cw-9_5-darwin  gcc-3.2.3-linux  gcc-3.3.6-linux  gcc-3.3.6-linux  gcc-3.4.4-linux  gcc-3.4.5-linux  gcc-3_3-darwin  gcc-4.0.2-linux  gcc-4_0-darwin  intel-9.0-linux  intel-win32-8_1  intel-win32-9_0  qcc-3.3.5-cpp  tru64cxx71-006  vc-7_1  vc-7_1  vc-7_1

Any chance that these two are going to be addressed? These tests keep
failing spirit on nearly all supported toolsets for quite some time now.

Markus



-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
Spirit-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/spirit-devel
Reply | Threaded
Open this post in threaded view
|

Re: [spirit] Re: Boost regression notification (2006-02-24 [HEAD])

Stefan Slapeta
Stefan Slapeta wrote:

> Fixed.
>

Not completely. I didn't see that we also need a handling for non-pod
scanner types and that doesn't pop up in the regressions for vc and
intel! (I should have tested CW as well...)

Joel, you fixed that some time ago (in december) and Hartmut reverted
this fix again:

Revision : 1.17
Date : 2006/1/10 23:30:24
Author : 'hkaiser'
State : 'Exp'
Lines : +26 -87
Description :
Reverted the recent patch, because it made more problems than goods.


Hartmut didn't answer my private mail yet, however, as Joel's code looks
fine to me, does what it is supposed to do and doesn't cause *any*
regression on the compilers I tested (vc, Intel, cw), I would suggest to
'reanimate' this version. If Hartmut has particular problems, we should
address these on this code base.

Stefan



-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
Spirit-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/spirit-devel
Reply | Threaded
Open this post in threaded view
|

Re: [spirit] Re: Boost regression notification (2006-02-24 [HEAD])

Stefan Slapeta
Stefan Slapeta wrote:

[...]

Ok, Hartmut has just informed me that it was reverted because of
http://tinyurl.com/r52fv and http://tinyurl.com/mgtcy.

It seems that this compiler isn't able to handle SFINAE at all??

Stefan



-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
Spirit-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/spirit-devel
Reply | Threaded
Open this post in threaded view
|

Re: [spirit] Re: Boost regression notification (2006-02-24 [HEAD])

Stefan Slapeta
In reply to this post by Stefan Slapeta
Stefan Slapeta wrote:

[...]

ok, fixed now. I assume that ?: is enough to inhibit instantation of
to_int_type and to_char_type if not appropriate. At least CW doesn't
complain anymore. I check it in and wait what other compilers say. The
worst case could be that we need separate implementations of isalpha_
etc. for pod and non-pod and activate them alternatively by enable_if.

I also extended the testcase in order to test all the other primitives, too.

IMO the behaviour for non-pod types is underspecified: currently we
require a conversion to a type that can be handled by isalpha etc. We
are currently *not* able to handle the same conversions for the
wide-char equivalents. This would require a concept that is completely
different from what we have now.

Stefan



-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
Spirit-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/spirit-devel