Hi Travis,

On Thu, Oct 21, 2010 at 8:30 AM, Travis Davies <[email protected]> wrote:
>
> Hi Guys,
>
> I am working on Bug 621870 and hoped you might give me some advice about the
> best way to do this.
> I am thinking of converting the 'st_sql_list' structure (in
> drizzled/sql_list.h) to c++ class using the STL list data structure as
> a private member.
> Like:
>
> #include <list>
> typedef class SQL_LIST
> {
> list<char> sql_list;
> public:
>    void push_front(...);
>    void push_back(...);
>    ...
> };
>
> Is this the approach you would take?

Thanks for taking this on!

Why choose this approach? Can we not just get rid of SQL_LIST
altogether and simply use std::list in its place? And I'm fairly certain
that list<char> isn't going to be the right type. I believe the idea was
that this data structure could be used to have a list of elements of
any type. Take a look at its use in drizzled/sql_lex.cc (add_to_list()
function) for an example.

-Dave

_______________________________________________
Mailing list: https://launchpad.net/~drizzle-discuss
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~drizzle-discuss
More help   : https://help.launchpad.net/ListHelp

Reply via email to