ЛА
Убираешь лок - ругается
Size: a a a
2020 April 08
ЛА
Вывод - дело не в локе, видимо
ВС
Разве?
Видимо перепутал, да, передавать ссылку не надо
EK
вот полный код для теста. Почему тогда в данном случае рейс есть? https://play.golang.org/p/tirFdsR3CSd
EK
может целое число нельзя лочить? так как оно по значению?
ЛА
Код мудреный
ВС
А, так считывание errorcount надо тоже под мьютекс
EK
А, так считывание errorcount надо тоже под мьютекс
где именно?
ВС
33 строка (if errorCount...)
EK
почему? race же вот здесь: errorCount = errorCount+1
ВС
Там отчёт немного мудреный, он показывает все места, где есть взаимодействие с переменной
EK
а как в if мутекс поставить?
EK
весь if обернуть им что ли?
ЛА
33 строка (if errorCount...)
+
ЛА
Это фиксит проблему
ВС
а как в if мутекс поставить?
можно что-то такое:
m.Lock()
localErrCount := errorCount
m.Unlock()
if localErrCount >= maxErrors {
ВС
Но намного проще просто переехать на atomic для такого
EK
спасибо помогло
ЛА
Но намного проще просто переехать на atomic для такого
Имхо, атомик не поменяет сути. Все равно в основном треде тоже нужно лок юзать
ЛА
Даже если в горутинах будет атомик