Skip to content

Commit b499fe3

Browse files
authored
Crash Fix (#290)
* Fix crash when setting cpu affinity to all cores if you have 32+ cores. For example setting cpu affinity to 1 core, then clicking All Processors -> Set. Prior it crashed when the last core listed with 32 CPU available was checked.
1 parent c7dc0d0 commit b499fe3

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

BardMusicPlayer/Controls/BardExtSettingsWindow.xaml.cs

+4-4
Original file line numberDiff line numberDiff line change
@@ -163,14 +163,14 @@ private void PopulateCPUTab()
163163

164164
private void Save_CPU_Click(object sender, RoutedEventArgs e)
165165
{
166-
long mask = 0;
166+
ulong mask = 0;
167167
var idx = 0;
168168
foreach (var box in _cpuBoxes)
169169
{
170170
if (box.IsChecked != null && (bool)box.IsChecked)
171-
mask += 0b1 << idx;
171+
mask += 0b1ul << idx;
172172
else
173-
mask += 0b0 << idx;
173+
mask += 0b0ul << idx;
174174
idx++;
175175
}
176176
//If mask == 0 show an error
@@ -181,7 +181,7 @@ private void Save_CPU_Click(object sender, RoutedEventArgs e)
181181
return;
182182
}
183183
else
184-
_performer.game.SetAffinity(mask);
184+
_performer.game.SetAffinity((long)mask);
185185
}
186186

187187
private void Clear_CPU_Click(object sender, RoutedEventArgs e)

0 commit comments

Comments
 (0)