>> > I am not convinced  this is the best way to solve the problem, or that
>> > we must even merge it. Yes, it is a problem, but that is just a hack.
>> > Can we do better?
>> >
>> > Dhaval
>> >
>> >
>> yes, the solution seems not optimal. Is it better to do the check in function
>> cg_set_control_value()? But if do this, the parameter list of this function
>> should be modified.
>>
>> Weng Meiling
>> Thanks!
>>
>>
> Yes, It is a bit hacky again. But it is improvement and I can't imagine 
> better way without major changes. Thus cote for this solution.
> Whay is your opinion Dhaval?

What worries is me is that that code looks very fragile. Also,
honestly, api.c has become a complete mess. Reading the code is really
hard. We really need to add more comments :-)

Let me think a bit more about it, and I will get back to you.

Dhaval

------------------------------------------------------------------------------
Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
Discover the easy way to master current and previous Microsoft technologies
and advance your career. Get an incredible 1,500+ hours of step-by-step
tutorial videos with LearnDevNow. Subscribe today and save!
http://pubads.g.doubleclick.net/gampad/clk?id=58040911&iu=/4140/ostg.clktrk
_______________________________________________
Libcg-devel mailing list
Libcg-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libcg-devel

Reply via email to