[PATCH] Demonstrate failing rtstring ctor

Attached patches add a test to OCL v1 to create rtstring instances in scripting. The test fails to use rtstring ctor, and instead attempts to create a string (ie std::string) first and then create rstring from that. This is bad.

I can't figure out what the problem is, and hope someone can shed some light.

Test implementation based on rtt/tests/state_test.cpp

Peter, this is based on the plugins/rtalloc that you pulled from my github into rtt-2.0-mainline sometime ago. But as that isn't in v1 nor v2 masters, I can only give you a test that works on my github repo. I can't base it on anything from your git* AFAICT.

Cheers
Stephen

AttachmentSize
0001-plugins-rtalloc-Add-test-case-demonstrating-failing-.patch15.67 KB
0002-plugins-rtalloc-Preliminary-implementation-of-rtstri.patch3.06 KB

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789

[PATCH] Demonstrate failing rtstring ctor

On Oct 18, 2010, at 11:23 , Peter Soetens wrote:

> On Tuesday 21 September 2010 13:19:15 Stephen Roderick wrote:
>> On Sep 21, 2010, at 03:26 , Peter Soetens wrote:
>>> On Tuesday 21 September 2010 02:55:22 S Roderick wrote:
>>>> Attached patches add a test to OCL v1 to create rtstring instances in
>>>> scripting. The test fails to use rtstring ctor, and instead attempts to
>>>> create a string (ie std::string) first and then create rstring from
>>>> that. This is bad.
>>>
>>> That's probably because std::string is hardcoded in the parsers, and it
>>> assumes every char* is going into a datasource of std::string. The only
>>> solution seems to be to pass it as a char* into a registered constructor
>>> for rt_string if a user writes it like this : rt_string("foo"), just
>>> like in C++.
>>
>> My test case has an example exactly like that, I believe. It fails.
>
> Could you add this as a bug report ? I'll loose track of it otherwise...
>
> Thanks,
>
> Peter

np

https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=789