Github user asfgit closed the pull request at:
https://github.com/apache/trafficserver/pull/286
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user AZNova commented on the pull request:
https://github.com/apache/trafficserver/pull/286#issuecomment-137756663
Updated the commit. Moved the swap() above its initial usage. Clarified
comments. Included amc's updated test_vec.cc which calls the new qsort as part
of regres
Github user SolidWallOfCode commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/286#discussion_r38672458
--- Diff: lib/ts/Vec.h ---
@@ -964,41 +965,71 @@ qsort_Vec(C *left, C *right, bool (*lt)(C, C))
}
}
} else {
-
Github user SolidWallOfCode commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/286#discussion_r38668014
--- Diff: lib/ts/Vec.h ---
@@ -964,41 +965,71 @@ qsort_Vec(C *left, C *right, bool (*lt)(C, C))
}
}
} else {
-
Github user AZNova commented on the pull request:
https://github.com/apache/trafficserver/pull/286#issuecomment-137493566
I have updated this pull request with a new commit. I ran all of the
regression tests using 'make check', and all passed. I have tested this with
approx 10k cert
Github user jpeach commented on the pull request:
https://github.com/apache/trafficserver/pull/286#issuecomment-136489609
I ran the regression tests on OS X with this patch, and saw some [memory
leaks](http://apaste.info/z1O). the ```test_certlookup``` test (part of ```make
check```)
GitHub user AZNova opened a pull request:
https://github.com/apache/trafficserver/pull/286
TS 3867 - Improved qsort
Updated qsort algo to a median of 3 qsort.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/AZNova/trafficserver TS
