From 27d502612c2ec5f68e0c5f5b9e7f851e3fa1acc9 Mon Sep 17 00:00:00 2001 From: xzyfer Date: Tue, 27 Mar 2018 17:33:26 +1100 Subject: [PATCH] Remove raw css imports This is a non-standard, and confusion feature. Implementors can now opt into supporting @import'ing additional file extentions with `sass_option_push_import_extension`. --- src/file.cpp | 18 +++--------------- src/file.hpp | 5 ++--- 2 files changed, 5 insertions(+), 18 deletions(-) diff --git a/src/file.cpp b/src/file.cpp index 61b8f2ddf..ab2065194 100644 --- a/src/file.cpp +++ b/src/file.cpp @@ -323,7 +323,7 @@ namespace Sass { // (2) underscore + given // (3) underscore + given + extension // (4) given + extension - std::vector resolve_includes(const std::string& root, const std::string& file, const std::vector& exts, const std::vector& d_exts) + std::vector resolve_includes(const std::string& root, const std::string& file, const std::vector& exts) { std::string filename = join_paths(root, file); // split the filename @@ -342,25 +342,13 @@ namespace Sass { for(auto ext : exts) { rel_path = join_paths(base, "_" + name + ext); abs_path = join_paths(root, rel_path); - if (file_exists(abs_path)) includes.push_back({{ rel_path, root }, abs_path }); + if (file_exists(abs_path)) includes.push_back({{ rel_path, root }, abs_path, ext == ".css" }); } // next test plain name with exts for(auto ext : exts) { rel_path = join_paths(base, name + ext); abs_path = join_paths(root, rel_path); - if (file_exists(abs_path)) includes.push_back({{ rel_path, root }, abs_path }); - } - // next test d_exts plus underscore - for(auto ext : d_exts) { - rel_path = join_paths(base, "_" + name + ext); - abs_path = join_paths(root, rel_path); - if (file_exists(abs_path)) includes.push_back({{ rel_path, root }, abs_path, true }); - } - // next test plain name with d_exts - for(auto ext : d_exts) { - rel_path = join_paths(base, name + ext); - abs_path = join_paths(root, rel_path); - if (file_exists(abs_path)) includes.push_back({{ rel_path, root }, abs_path, true }); + if (file_exists(abs_path)) includes.push_back({{ rel_path, root }, abs_path, ext == ".css" }); } // nothing found return includes; diff --git a/src/file.hpp b/src/file.hpp index 718a529cd..a043bea7a 100644 --- a/src/file.hpp +++ b/src/file.hpp @@ -127,11 +127,10 @@ namespace Sass { namespace File { static std::vector defaultExtensions = { ".scss", ".sass" }; - static std::vector deprecatedExtensions = { ".css" }; std::vector resolve_includes(const std::string& root, const std::string& file, - const std::vector& exts = defaultExtensions, - const std::vector& d_exts = deprecatedExtensions); + const std::vector& exts = defaultExtensions); + }