Re: Recursive SymbolNames solved.

[Basile B. via Digitalmars-d-announce]

On Wednesday, 8 June 2016 at 20:15:57 UTC, FlatBareRunner wrote:

On Wednesday, 8 June 2016 at 19:41:49 UTC, deadalnix wrote:

On Wednesday, 8 June 2016 at 13:28:19 UTC, Stefan Koch wrote:

Hi, I solved the issue.
PR is coming shortly.


Dude come on, that isn't an announce. There is No PR, there is 
no description of the solution, there is nothing.


Hey guys, I cured cancer and solved world hunger. PR coming 
soon.


Lol ! My turn:

I'm a genious but I haven't done anything genial yet...


Writer here. Not yet published but working hard on the first 
one...

Re: Recursive SymbolNames solved.

[Rainer Schuetze via Digitalmars-d-announce]

On 09.06.2016 11:02, Johan Engelen wrote:

On Thursday, 9 June 2016 at 04:58:45 UTC, Stefan Koch wrote:

On Wednesday, 8 June 2016 at 13:28:19 UTC, Stefan Koch wrote:

Hi, I solved the issue.
PR is coming shortly.


Solution is as follows:

Keep a list of already visited symbols in the mangler.
And information where we store the mangle.
meaning the position in the mangle string.
If a symbol is encountered for the second time store a backrefernce
rather then the symbol name again.


You mean this?
https://github.com/weka-io/ldc/commit/ffd6b55a4b83b3007d9690cbd9a25d07b243a00e



This doesn't handle the manglers created locally for templates and 
parameter tuples. Here's a more complete version: 
https://github.com/dlang/dmd/pull/5855

Re: Recursive SymbolNames solved.

[Johan Engelen via Digitalmars-d-announce]

On Thursday, 9 June 2016 at 04:58:45 UTC, Stefan Koch wrote:

On Wednesday, 8 June 2016 at 13:28:19 UTC, Stefan Koch wrote:

Hi, I solved the issue.
PR is coming shortly.


Solution is as follows:

Keep a list of already visited symbols in the mangler.
And information where we store the mangle.
meaning the position in the mangle string.
If a symbol is encountered for the second time store a 
backrefernce rather then the symbol name again.


You mean this?
https://github.com/weka-io/ldc/commit/ffd6b55a4b83b3007d9690cbd9a25d07b243a00e

It did not result in a large reduction of symbol sizes.
http://forum.dlang.org/post/szodxrizfmufqdkpd...@forum.dlang.org

Re: Recursive SymbolNames solved.

[Stefan Koch via Digitalmars-d-announce]

On Wednesday, 8 June 2016 at 13:28:19 UTC, Stefan Koch wrote:

Hi, I solved the issue.
PR is coming shortly.


Solution is as follows:

Keep a list of already visited symbols in the mangler.
And information where we store the mangle.
meaning the position in the mangle string.
If a symbol is encountered for the second time store a 
backrefernce rather then the symbol name again.

Re: Recursive SymbolNames solved.

[FlatBareRunner via Digitalmars-d-announce]

On Wednesday, 8 June 2016 at 19:41:49 UTC, deadalnix wrote:

On Wednesday, 8 June 2016 at 13:28:19 UTC, Stefan Koch wrote:

Hi, I solved the issue.
PR is coming shortly.


Dude come on, that isn't an announce. There is No PR, there is 
no description of the solution, there is nothing.


Hey guys, I cured cancer and solved world hunger. PR coming 
soon.


Lol ! My turn:

I'm a genious but I haven't done anything genial yet...

Re: Recursive SymbolNames solved.

[FlatBareRunner via Digitalmars-d-announce]

On Wednesday, 8 June 2016 at 18:02:41 UTC, Jerry wrote:

On Wednesday, 8 June 2016 at 13:28:19 UTC, Stefan Koch wrote:

Hi, I solved the issue.
PR is coming shortly.


How was they getting recursive?


Think to the fibo templatized example, it generates a very long 
mangle with basically always the same symbol.

Re: Recursive SymbolNames solved.

[Jerry via Digitalmars-d-announce]

On Wednesday, 8 June 2016 at 13:28:19 UTC, Stefan Koch wrote:

Hi, I solved the issue.
PR is coming shortly.


How was they getting recursive?

Recursive SymbolNames solved.

[Stefan Koch via Digitalmars-d-announce]

Hi, I solved the issue.
PR is coming shortly.