Re: [boost] [ boost ] [ Trie ]

Subject: Re: [boost] [ boost ] [ Trie ]
From: Cosmin Boaca (boost.cosmin.boaca_at_[hidden])
Date: 2015-02-20 13:21:10

Hello,

Finally I have managed to run the tests.

Cosmin

On 20 February 2015 at 18:11, Cosmin Boaca <boost.cosmin.boaca_at_[hidden]>
wrote:

> Hello,
>
> I didn't managed to run the tests on Windows using bjam and also couldn't
> manage to run any kind of boost test in visual studio. Any test I am trying
> to run receveies acces violation error .
>
> Cosmin
>
> On 20 February 2015 at 11:58, Antony Polukhin <antoshkka_at_[hidden]> wrote:
>
>> 2015-02-20 13:36 GMT+04:00 Cosmin Boaca <boost.cosmin.boaca_at_[hidden]>:
>> <...>
>>
>> > However, I do not want do this during GSoC because I won't be available
>> > during the summer. I want to contribute during my spare time.
>> >
>>
>> This sounds like a killer feature to me :) I'd gladly help you.
>>
>> As I see, you've already started the work on Trie project by removing the
>> Compare template parameter from trie_node. That's a good start!
>>
>> Lets continue the code simplification. Looks like trie_reverse_iterator
>> could be removed and std::reverse_iterator used instead. This will reduce
>> the code size and simplify the maintainability.
>>
>> clear() and destroy() functions almost duplicate each other. Looks like
>> clear() must work as destroy(), so fix the clear() method and use it
>> everywhere in code instead of destroy(). Make sure that everything works
>> well, and tests pass.
>>
>> In case of trouble or finishing the task - notify me.
>>
>> Good luck! Hope you'll enjoy the project!
>>
>> --
>> Best regards,
>> Antony Polukhin
>>
>> _______________________________________________
>> Unsubscribe & other changes:
>> http://lists.boost.org/mailman/listinfo.cgi/boost
>>
>
>