Skip to content

Commit 3d61646

Browse files
committed
Disable unused sass 4.0 code-paths
1 parent 2c4b0c3 commit 3d61646

File tree

2 files changed

+15
-12
lines changed

2 files changed

+15
-12
lines changed

src/extender.cpp

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,9 @@ namespace Sass {
251251
// ##########################################################################
252252
// Helper function to copy extension between maps
253253
// ##########################################################################
254-
void Extender::mapCopyExts(
254+
// Seems only relevant for sass 4.0 modules
255+
// ##########################################################################
256+
/* void mapCopyExts(
255257
ExtSelExtMap& dest,
256258
const ExtSelExtMap& source)
257259
{
@@ -271,7 +273,7 @@ namespace Sass {
271273
}
272274
}
273275
}
274-
}
276+
} */
275277
// EO mapCopyExts
276278

277279
// ##########################################################################
@@ -353,11 +355,13 @@ namespace Sass {
353355
auto existingExtensions = extensionsByExtender.find(target);
354356
if (existingExtensions != extensionsByExtender.end()) {
355357
if (hasExistingExtensions && !existingExtensions->second.empty()) {
356-
auto additionalExtensions =
358+
// Seems only relevant for sass 4.0 modules
359+
// auto additionalExtensions =
357360
extendExistingExtensions(existingExtensions->second, newExtensionsByTarget);
358-
if (!additionalExtensions.empty()) {
361+
// Seems only relevant for sass 4.0 modules
362+
/* if (!additionalExtensions.empty()) {
359363
mapCopyExts(newExtensionsByTarget, additionalExtensions);
360-
}
364+
} */
361365
}
362366
}
363367

@@ -455,6 +459,8 @@ namespace Sass {
455459
}
456460
else {
457461
sources.insert(complex, withExtender);
462+
/*
463+
// Seems only relevant for sass 4.0 modules
458464
for (auto& component : complex->elements()) {
459465
if (auto compound = component->getCompound()) {
460466
for (auto& simple : compound->elements()) {
@@ -465,15 +471,19 @@ namespace Sass {
465471
if (newExtensions.find(extension.target) != newExtensions.end()) {
466472
additionalExtensions[extension.target].insert(complex, withExtender);
467473
}
474+
*/
468475
}
469476
}
470477

471478
// If [selectors] doesn't contain [extension.extender],
472479
// for example if it was replaced due to :not() expansion,
473480
// we must get rid of the old version.
481+
/*
482+
// Seems only relevant for sass 4.0 modules
474483
if (!containsExtension) {
475484
sources.erase(extension.extender);
476485
}
486+
*/
477487

478488
}
479489

src/extender.hpp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -249,13 +249,6 @@ namespace Sass {
249249
const Extension& lhs,
250250
const Extension& rhs);
251251

252-
// ##########################################################################
253-
// Helper function to copy extension between maps
254-
// ##########################################################################
255-
static void mapCopyExts(
256-
ExtSelExtMap& dest,
257-
const ExtSelExtMap& source);
258-
259252
// ##########################################################################
260253
// Extend [extensions] using [newExtensions].
261254
// ##########################################################################

0 commit comments

Comments
 (0)