[Kamailio-Devel] [ openser-Patches-2665569 ] db_text support for order-by clause, uses qsort

[SourceForge.net]

Patches item #2665569, was opened at 2009-03-05 15:59
Message generated for change (Comment added) made by henningw
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=2665569&group_id=139143

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: ver devel
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Edgar Holleis (ccaracalla)
Assigned to: Nobody/Anonymous (nobody)
Summary: db_text support for order-by clause, uses qsort

Initial Comment:
- parses db_key_t _o as if it was an SQL-fragment
- sort by multiple columns supported
- SQL keywords ASC and DESC supported
- supports sorting by columns that would not normally be retrieved
- removes extra columns before returning result
- uses C-libraries qsort function
- patch is against 1.5.0, but patch against 1.3.* also available
- patch includes fixes for issues #2658723 and #2658593

----------------------------------------------------------------------

>Comment By: Henning Westerholt (henningw)
Date: 2009-03-06 23:49

Message:
Hi Edgar,

thanks for the patch, it looks really good. The only thing that i noticed
is the usage of setjmp/ longjmp for the error handling, which is not really
used in our code base. Perhaps we can find a alternative solution for this,
what do you think?

Henning

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=2665569&group_id=139143