>> > 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