-
Notifications
You must be signed in to change notification settings - Fork 492
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
IndexStore
: Make SmartHeaderChain
property private
#11378
IndexStore
: Make SmartHeaderChain
property private
#11378
Conversation
@@ -251,7 +251,7 @@ public async Task AddNewFiltersAsync(IEnumerable<FilterModel> filters) | |||
{ | |||
using (await IndexLock.LockAsync(CancellationToken.None).ConfigureAwait(false)) | |||
{ | |||
await using SqliteTransaction sqliteTransaction = IndexStorage.BeginTransaction(); | |||
using SqliteTransaction sqliteTransaction = IndexStorage.BeginTransaction(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
await
is not necessary here (it emits a warning) because async is not supported by sqlite (https://learn.microsoft.com/en-gb/dotnet/standard/data/sqlite/async)
@@ -70,7 +70,7 @@ public IndexStore(string workFolderPath, Network network, SmartHeaderChain smart | |||
/// <summary>Run migration if SQLite file does not exist.</summary> | |||
private bool RunMigration { get; } | |||
|
|||
public SmartHeaderChain SmartHeaderChain { get; } | |||
private SmartHeaderChain SmartHeaderChain { get; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Main change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cACK and code LGTM
This is more correct way to pass dependencies.