
Alexander
16.10.2018
22:47:30
именно!
между прочим у меня там нету сплошного unlifted ^_^
у DMap пока вставка круче :/
но видимо бенчмарки плохие

Google

Alexander
16.10.2018
22:49:11
и там удачный случай
ещё круто бы было все с liquid haskell пройтись
@int_index кстати мне не нравится то, что у вас 2 fingerprint массива
ты пробовал в одном все хранить друг за другом? и умножать на 2 индекс для сдвига

Index
16.10.2018
22:51:11
Именно так и пробовали сначала, вышло медленнее

Alexander
16.10.2018
22:51:13
должно быть более cache friendly (но это не точно)

Index
16.10.2018
22:51:55
Это изначальная идея была, но unboxed vectors оказались быстрее, и я подсмотрел как там сделано — разбиение на два

Alexander
16.10.2018
22:52:18
страннота
ещё на самом деле, но это не точно, можно хранить только половинку слова и сравнивать, как в hashmap
блин слово в смысле
а дальше хранить smallarray## второе слово+ значение
но в принципе и так все хорошо уже

Dmitry
17.10.2018
00:17:33

Google

Dmitry
17.10.2018
04:40:08
@chshersh чот у вас слишком простые ревью, я повеселее подкинул
Ну, мне не особо нравится не соблюдение нашего стайл-гайда, но не очень удобно говорить про такое, особенно когда человек потратил много времени и сил и забесплатно поделал клевые штуки в твоих библиотеках. Форматирование я потом и сам могу поправить, когда буду с этим кодом работать, и вообще, не столько важны отступы, сколько именно смысл. А если в остальном замечаний к PR не хватает — извините, под конец дня не особо сил остается ? Там просто очень много делается, и на это надо больше времени, чтобы отвревьювить. Тем более, я не особо ожидал, что во время Hacktoberfest столько людей будет что-то делать и будут открывать PR, поэтому наши силы касательно ревью PR потихоньку истекают...

Dmitry
17.10.2018
06:23:16

Alexander
17.10.2018
06:26:46
a -> b ~ b^a разве нет?

Oleg
17.10.2018
06:27:19

Alexander
17.10.2018
06:27:23
Ну, мне не особо нравится не соблюдение нашего стайл-гайда, но не очень удобно говорить про такое, особенно когда человек потратил много времени и сил и забесплатно поделал клевые штуки в твоих библиотеках. Форматирование я потом и сам могу поправить, когда буду с этим кодом работать, и вообще, не столько важны отступы, сколько именно смысл. А если в остальном замечаний к PR не хватает — извините, под конец дня не особо сил остается ? Там просто очень много делается, и на это надо больше времени, чтобы отвревьювить. Тем более, я не особо ожидал, что во время Hacktoberfest столько людей будет что-то делать и будут открывать PR, поэтому наши силы касательно ревью PR потихоньку истекают...
блин, я забыл, что не у всех наш стиль

Dmitry
17.10.2018
06:29:22

Index
17.10.2018
07:27:06
> мне не особо нравится не соблюдение нашего стайл-гайда
Зато в других проектах нравится применять stylish-haskell, когда этого никто не просил


Dmitry
17.10.2018
07:48:34
> мне не особо нравится не соблюдение нашего стайл-гайда
Зато в других проектах нравится применять stylish-haskell, когда этого никто не просил
Во-первых, это было давно, и я так больше не делаю.
Во-вторых, я научился отключать stylish-haskell, когда работаю над проектом (правда, это неудобно).
В-третьих, поскольку это больно, то мотивации работать над проектами, которые не используют stylish-haskell, у меня намного меньше, чем над теми, которые используют. Слишком много сил тратится на переключение контекста.
В-четвертых, если даже проект и использует stylish-haskell, то я считаю, что настройки должны лежать в корне этого проекта (а так делают не все люди, к сожалению).
В-пятых, проблема часто еще и в том, что у людей не просто stylish-haskell не настроен, но и какой-нибудь whitespace cleaner тоже не настроен, и ты коммитишь с удалением trailing пробелов. А потом люди жалуются, что коммит содержит лишние изменения пробелов (и не очень понятно, почему нельзя использовать GitHub фичу для игнорирования пробельных изменений в ревью). Это, кстати, была одна из причин для меня перестать контрибьютить в cabal. Я не нашел способа отключить у себя whitespace cleaner, а мне лично самому не особо и нужно.
Сам же я спокойно отношусь к спонтанным форматированиям и выравниваям в PR, если они в рамках настроек и стайл-гайда и здравого смысла.


Pineapple
17.10.2018
07:52:16
> удалением лидирующих пробелов
Это как?

Dmitry
17.10.2018
07:54:57

Index
17.10.2018
07:56:30
Такую зависимость от стиля надо бороть, я вот с GHC исходниками работаю и со странным выравниванием, и с их do {a; b}
И не бросаюсь все переписывать
Просто беру и такой же странный код пишу, а в своих проектах у меня свой стиль

kana
17.10.2018
07:59:56
нужно или ненужно бороть - это спорный момент, факт в том, разностилевой код - это проблема, которую нужно решать (игнорирование - тоже частично решение, конечно). Например, введенем общего стиля (такое себе решение на самом деле), или редактором, который отображает код с форматирование под тебя, а в самом коде форматирования нет вовсе (или хранится аст) (вроде был один такой)

Dmitry
17.10.2018
07:59:58

Index
17.10.2018
08:00:37
Так говно тулы, если нельзя их научить

Leonid
17.10.2018
08:01:11
люди коммитящие с висящими пробелами должны гореть в аду

Index
17.10.2018
08:01:31
Там же где люди удаляющие их без нужды
У них там один котёл

Alexander
17.10.2018
08:02:11

Google

Alexander
17.10.2018
08:02:15
мне лень вим таки настроить

Pavel
17.10.2018
08:02:18

Алексей
17.10.2018
08:02:29
Это типа до сих пор не выработался единый стиль форматирования?

Leonid
17.10.2018
08:02:36
я!
это всё тлетворное влияние физматшколы

kana
17.10.2018
08:02:40
а нельзя сделать просто один коммит на весь ghc/cabal/еще что, который удалит все висящие пробелы (локализует катастрофу)

Pineapple
17.10.2018
08:02:44

Alexander
17.10.2018
08:03:02
общий стиль есть только у новых языков где стилизатор встроен в компилятор или рядом

Pineapple
17.10.2018
08:03:03

Alexander
17.10.2018
08:03:08
типа го и раста

Leonid
17.10.2018
08:03:11

Alexander
17.10.2018
08:03:16
больше ни у кого общего стиля нету

Leonid
17.10.2018
08:03:45
я вот в линукс не контрибучу может потому что у них табы для отступов

Алексей
17.10.2018
08:03:57

Дед Пегас
17.10.2018
08:04:03
Даёшь 8 пробелов отступы.

Алексей
17.10.2018
08:04:11
В питоне пеп8 есть к примеру

Alexander
17.10.2018
08:05:07
вообще тут сложно договориться красивенько против поддерживаеменько

Алексей
17.10.2018
08:05:11
Хорошо, если ослабить требование прям совсем: хотя бы несколько популярных стилей форматирования есть или каждый как хочет так и форматирует?

kana
17.10.2018
08:05:51
по моему нет даже общепринятых

Алексей
17.10.2018
08:06:32
это плохо

Pineapple
17.10.2018
08:06:55
Ну вроде стиль Йохана Тиббела ±широко распространён(с вариациями)

Google

A64m
17.10.2018
08:09:56
хорошо
меньше оснований требовать друг от друга какой-то бессмысленной ерунды

IC
17.10.2018
08:11:00

Pineapple
17.10.2018
08:11:36

Алексей
17.10.2018
08:11:46

Pineapple
17.10.2018
08:11:48
Но откровенно наркоманские стили — редкость всё же

A64m
17.10.2018
08:12:12

Алексей
17.10.2018
08:12:39

A64m
17.10.2018
08:13:53
В языках с нормальным синтаксисом плохо отформатировать - надо сильно постараться, никто, кроме крисдона не старается, так что все читабельно

Алексей
17.10.2018
08:14:35

A64m
17.10.2018
08:14:52
чет теоретик разбушевался

Pineapple
17.10.2018
08:16:07
А маршировать строем практики любят? eine Sprache, ein Stil, ein Editor!

Pavel
17.10.2018
08:20:54
пора заводить haskell_drama канал

Алексей
17.10.2018
08:23:23
чет теоретик разбушевался
Некоторые хаскелисты любят писать
data Record = Record {
field1 :: Int
, field2 :: String
}
После такого любой здравомыслящий человек разбушуется

Pineapple
17.10.2018
08:24:01
Да, это из более наркоманских стилей

Дед Пегас
17.10.2018
08:24:10

Алексей
17.10.2018
08:24:31

Дед Пегас
17.10.2018
08:24:45
Просто мне тут только отступа в 8 пробелов не хватало.

Алексей
17.10.2018
08:25:07
А конкретно?
ну да
, я был не прав
, всё нормально с этим стилем

Дед Пегас
17.10.2018
08:26:04
Вот так ещё можно.
data Record = Record {
field1 :: Int
,
field2 :: String
}

Google

Алексей
17.10.2018
08:26:31

Александр
17.10.2018
08:27:20
В Хаскеле типы - это самое важное.
data Record = Record { field1
:: Int, field2
:: String
}

Дед Пегас
17.10.2018
08:27:59

Алексей
17.10.2018
08:28:21

Yuriy
17.10.2018
08:30:07

Алексей
17.10.2018
08:30:23
а точно, простите

Pineapple
17.10.2018
08:30:28

Алексей
17.10.2018
08:32:56
Такой вопрос. Какой грязный извращенец придумал переносить запятую на новую строчку и что он перед этим употреблял?

Leonid
17.10.2018
08:33:17

Pineapple
17.10.2018
08:33:48

Alexander
17.10.2018
08:34:03

Андрей
17.10.2018
08:34:13

Pineapple
17.10.2018
08:34:19
Считайте это частью общепринятого стиля

Alexander
17.10.2018
08:34:32
устойчиво к диффам и рефакторингу и добавлением ещё конструкторов
вот запятая впереди - точно общий стиль

Leonid
17.10.2018
08:35:01
я бы во всех яп переносил запятую, но лень ковырятся в индентации в емаксах

Дед Пегас
17.10.2018
08:35:14

Alexander
17.10.2018
08:35:33
и выравнивается удобно

Dmitry
17.10.2018
08:35:41

Alexander
17.10.2018
08:35:52
не надо смотреть на конец чтобы увидеть есть ли запятая