These are chat archives for synrc/n2o

15th
Jul 2018
Pavel Kozlovsky
@thecynicalpaul
Jul 15 2018 09:12
Здравствуйте. Я тут ваш код изучаю, и заметил, что вы заменили свою имплементацию в utf8.js на TextEncoder. (require "./te.js".) Возможно ли узнать, что это за энкодер по конкретнее?
Namdak Tonpa
@5HT
Jul 15 2018 10:19
положил
но не советую это использовать
это просто плейсхолдер для старых браузеров и фолбека
или как джаваскриптеры говорят полифила
вы туда должны подставить свою имплементацию для старых браузеров
https://github.com/synrc/n2o/blob/master/priv/te.js (пример, возможно не рабочий)
Pavel Kozlovsky
@thecynicalpaul
Jul 15 2018 10:23
Благодарю! Да я осматривал разные полифилы, но самые популярные страдают проблемами с памятью, так что вот ищю альтернативы. В данный момент Сафари и Эдж не поддерживают этот энкодер.
Namdak Tonpa
@5HT
Jul 15 2018 12:54
Так а это, что я знакомитил оно рабочее вообще?
Если найдёте подходящий вам полифил, присылайте пул рнквест, я перепишу вместо te.js
Sébastien Saint-Sevin
@seb3s
Jul 15 2018 12:56
Hi Namdak,
a few days ago, I've submitted a PR to mad repo
just to know if you plan to integrate this kind of changes or not :-)
fix sys.config include files
Namdak Tonpa
@5HT
Jul 15 2018 13:21
Let me check!
Did you get my Code Review I made immediately after PR ?
load_sysconfig(load_sysconfig()) isn't clear.

Could you refactor the whole module and reduce everything?
Namdak Tonpa
@5HT
Jul 15 2018 13:26
This module need to be definitely rewritten.
I can give you a chance :-) If you pass I will see what I can do in two days (possible full rewrite).
Is this ok for you ?
Sébastien Saint-Sevin
@seb3s
Jul 15 2018 13:27
I didn'get any code review
Namdak Tonpa
@5HT
Jul 15 2018 13:27
Just see my link I gave now.
Oh
sorry I missed SEND REVIEW button
MY BAD
Sébastien Saint-Sevin
@seb3s
Jul 15 2018 13:28
aha ok, I was lost :-)
Sébastien Saint-Sevin
@seb3s
Jul 15 2018 13:53
It could be renamed something like
Config = load_sysconfig(),
application_config(Config),
That would be clearer on what is done
Namdak Tonpa
@5HT
Jul 15 2018 16:50
Well I can give you a direct access!
Check invite!
Namdak Tonpa
@5HT
Jul 15 2018 17:42
Please make mad better! :-)