Enable Unicode support for SCSS Compiler
authorTim Düsterhus <duesterhus@woltlab.com>
Thu, 24 Jun 2021 13:33:09 +0000 (15:33 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Thu, 24 Jun 2021 13:38:29 +0000 (15:38 +0200)
The large performance difference measured back when this was introduced appears
to be gone. On PHP 8 both encodings appear to be equally fast. On PHP 7.4 the
UTF-8 version is ~25% slower, which is certainly not ideal, but not as bad as a
7× increase. In any case the UTF-8 version completes in way less than 2 seconds
which I consider to be the line that the CSS compiler should not cross.

Resolves #4339
see #4274

wcfsetup/install/files/lib/system/style/StyleCompiler.class.php

index c5c58707f0ba8b59dec761721ef392ef51a1243a..3bf88833817936208b51f1226249f215d7b2331a 100644 (file)
@@ -76,9 +76,6 @@ final class StyleCompiler extends SingletonFactory
     protected function makeCompiler(): Compiler
     {
         $compiler = new Compiler();
-        // Disable Unicode support because of its horrible performance (7x slowdown)
-        // https://github.com/WoltLab/WCF/pull/2736#issuecomment-416084079
-        $compiler->setEncoding('iso8859-1');
         $compiler->setImportPaths([WCF_DIR]);
 
         if (\ENABLE_DEBUG_MODE && \ENABLE_DEVELOPER_TOOLS) {