beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-21 15:10:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

AK: Make TemporaryChange not copy the old value twice

Browse source 
This is necessary for the next commit (and might even help performance
in some very weird cases).
This commit is contained in:
Ali Mohammad Pur 2024-10-10 09:57:42 +02:00 committed by Tim Ledbetter
parent 512cea5a87
commit e5f87eb12b
Notes: github-actions[bot] 2024-10-10 22:55:05 +00:00 
Author: https://github.com/alimpfard
Commit: https://github.com/LadybirdBrowser/ladybird/commit/e5f87eb12bd
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/1710
Reviewed-by: https://github.com/DanShaders ✅
Reviewed-by: https://github.com/tcl3 ✅
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
AK/TemporaryChange.h
View file

@ -15,11 +15,11 @@ class TemporaryChange {
public: public:
TemporaryChange(T& variable, T value) TemporaryChange(T& variable, T value)
: m_variable(variable) : m_variable(variable)
, m_old_value(variable) , m_old_value(move(variable))
{ {
m_variable = value; m_variable = move(value);
} }
~TemporaryChange() { m_variable = m_old_value; } ~TemporaryChange() { m_variable = move(m_old_value); }
private: private:
T& m_variable; T& m_variable;