| Summary | Merge tests for horde_lz4 and lzf in test.php |
| Queue | Horde Framework Packages |
| Type | Enhancement |
| State | Resolved |
| Priority | 1. Low |
| Owners | jan (at) horde (dot) org |
| Requester | arjen+horde (at) de-korte (dot) org |
| Created | 09/21/2017 (2847 days ago) |
| Due | |
| Updated | 09/25/2017 (2843 days ago) |
| Assigned | |
| Resolved | 09/25/2017 (2843 days ago) |
| Milestone | |
| Patch | No |
State ⇒ Resolved
commit 650f33e7e4259b3a35db7a3b2bad0970f8961a3d
Author: Jan Schneider <jan@horde.org>
Date: Mon Sep 25 17:36:41 2017 +0200
Improve check for horde_lz4/lzf (
Request #14693).horde/lib/Test.php | 27 +++++++++++++++++----------
1 file changed, 17 insertions(+), 10 deletions(-)
http://github.com/horde/horde/commit/650f33e7e4259b3a35db7a3b2bad0970f8961a3d
Priority ⇒ 1. Low
Patch ⇒ No
Milestone ⇒
Queue ⇒ Horde Framework Packages
Summary ⇒ Merge tests for horde_lz4 and lzf in test.php
Type ⇒ Enhancement
State ⇒ New
there is a preference to use the (faster) latter one. In test.php
there is a test for both separately and if only 'horde_lz4' is
installed, it still mentions 'lzf'.
It would be clearer if these tests would be merged into one, with a
suggestion that if only 'lzf' is installed, 'horde_lz4' would be a
better alternative and if neither is installed, only 'horde_lz4' is
recommended.