Skip to content

Commit 2b076d4

Browse files
author
Stewart Miles
committed
Fixed stack overflow when saving project settings.
Fixes googlesamples#304 Bug: 145695256 Change-Id: I153d8a085b834796546cae4ed160128f83c277d8
1 parent cd1b720 commit 2b076d4

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
# Version 1.2.135 - Dec 5, 2019
2+
## Bug Fixes
3+
* All Components: Fixed stack overflow when loading project settings.
4+
15
# Version 1.2.134 - Dec 4, 2019
26
## Bug Fixes
37
* All Components: Fixed an issue which caused project settings to be cleared

source/VersionHandlerImpl/src/ProjectSettings.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -741,7 +741,7 @@ public void DeleteKeys(IEnumerable<string> names) {
741741
/// <param name="save">Whether to save the settings.</param>
742742
internal static void DeleteAllProjectKeys(bool save = true) {
743743
lock (classLock) {
744-
LoadIfEmpty();
744+
if (save) LoadIfEmpty();
745745
foreach (var key in new List<string>(projectSettings.Keys)) {
746746
projectSettings.DeleteKey(key);
747747
}

0 commit comments

Comments
 (0)