Investigate memory error in mixins.less test case
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Hokwelum
	Dec 6 2023, 4:03 AM

Description

We got this error while running the less.js v2.5.3 mixins.less test case.

PHP Fatal error:  Allowed memory size of 134217728 bytes exhausted (tried to allocate 135168 bytes) in less.php/lib/Less/Environment.php on line 155

	Subject	Repo	Branch	Lines +/-
	MixinCall,Selector: Fix memory leak and selector bug	mediawiki/libs/less.php	master	+6 -5
	MixinDefinition,MixinCall: Minor clean up to match ref implementation	mediawiki/libs/less.php	master	+19 -40

Status	Assigned	Task
Open	None	T288498 Update less.php port to support Less.js 3.13 behaviours
Resolved	Hokwelum	T365481 Update less.php port to complete support Less.js 2.5.3 behaviours
Resolved	Krinkle	T352829 Investigate memory error in mixins.less test case

Krinkle triaged this task as Medium priority.Jan 5 2024, 2:22 PM

Change 988019 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/libs/less.php@master] MixinDefinition,MixinCall: Minor clean up to match ref implementation

Change 988020 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/libs/less.php@master] MixinCall,Selector: Fix memory leak and selector bug

Change 988019 merged by jenkins-bot:

[mediawiki/libs/less.php@master] MixinDefinition,MixinCall: Minor clean up to match ref implementation

Change 988020 merged by jenkins-bot:

[mediawiki/libs/less.php@master] MixinCall,Selector: Fix memory leak and selector bug

Krinkle closed this task as Resolved.Jan 9 2024, 3:06 PM