judofyr has quit [Remote host closed the connection]
mbj has quit [Ping timeout: 245 seconds]
mbj has joined #rom-rb
lgierth has quit [Quit: Ex-Chat]
dkubb has joined #rom-rb
<dkubb>
good morning
judofyr has joined #rom-rb
lfox has joined #rom-rb
<mbj>
dkubb: hola
<mbj>
dkubb: Pushed some code that removes the LSP violation in unparser.
<mbj>
dkubb: I'll not merge the current open PR and push refactorings with the same feature instead. LSP free.
lfox has quit [Client Quit]
<mbj>
dkubb: I expect we'll add an AST preprocessor soon. This one will make sure the AST carries all information we need to avoid backtracing at all.
<dkubb>
mbj: I still think it might be nicer to replace that conditional logic with polymorphism :P
snusnu has joined #rom-rb
<dkubb>
mbj: although I guess that class was kinda large that you replaced
<dkubb>
large for what it did I mean
<dkubb>
one thing I wish ruby had was multimethod dispatch with guard clauses. then you could handle logic like that with a few methods that have the same name, but different guard clauses
<dkubb>
it pushes the conditional logic into the language rather than making it explicit
judofyr has quit [Remote host closed the connection]
judofyr has joined #rom-rb
judofyr has quit [Ping timeout: 264 seconds]
judofyr has joined #rom-rb
lfox has joined #rom-rb
lfox has quit [Quit: ZZZzzz…]
<mbj>
dkubb: Heh, currently its better than before. But wait for the next commit.
<mbj>
We can just remove all special cases via preprocessing.
<mbj>
The ruby AST will be enriched with "virtual" nodes that have semantic and "emit-in-specific-style" information