Skip to content

Commit bfccbad

Browse files
committed
ATR-966: fixes for minor Claude remarks
1 parent e69a4bb commit bfccbad

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

src/Acuminator/Acuminator.Vsix/Coloriser/PXRoslynColorizerTagger.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,7 @@ internal partial class PXRoslynColorizerTagger : PXTaggerBase, ITagger<IClassifi
3434
public sealed override bool HasReferenceToAcumaticaPlatform => _hasReferenceToAcumaticaPlatform;
3535

3636
private readonly object _lastTaggingLock = new object();
37-
private volatile bool _lastTaggingWasSuccessful;
38-
39-
private volatile Workspace? _subscribedWorkspace;
40-
private readonly object _workspaceSubscriptionLock = new();
37+
private bool _lastTaggingWasSuccessful;
4138

4239
internal override bool LastTaggingWasSuccessful
4340
{
@@ -57,6 +54,9 @@ internal override bool LastTaggingWasSuccessful
5754
}
5855
}
5956

57+
private volatile Workspace? _subscribedWorkspace;
58+
private readonly object _workspaceSubscriptionLock = new();
59+
6060
private readonly RoslynWorkspaceProvider _roslynWorkspaceProvider;
6161

6262
public PXRoslynColorizerTagger(ITextBuffer buffer, PXColorizerTaggerProvider provider, bool subscribeToSettingsChanges,

src/Acuminator/Acuminator.Vsix/Coloriser/Roslyn/RoslynWorkspaceProvider.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,8 @@ private void InvokeWorkspaceChangedEventSafely(DocumentWorkspaceChangedEventArgs
8383
}
8484
catch (Exception exception)
8585
{
86-
AcuminatorVSPackage.Instance.AcuminatorLogger.LogException(exception, logOnlyFromAcuminatorAssemblies: false,
87-
LogMode.Warning);
86+
AcuminatorVSPackage.Instance?.AcuminatorLogger?.LogException(exception, logOnlyFromAcuminatorAssemblies: false,
87+
LogMode.Warning);
8888
}
8989
}
9090

0 commit comments

Comments
 (0)