Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Segmentation fault #42

Open
mkusher opened this issue May 18, 2016 · 7 comments
Open

Segmentation fault #42

mkusher opened this issue May 18, 2016 · 7 comments
Labels
Projects
Milestone

Comments

@mkusher
Copy link
Member

mkusher commented May 18, 2016

Padawan could not generate index for Symfony\Component\VarDumper\Tests\Fixtures\NotLoadableClass

@mkusher
Copy link
Member Author

mkusher commented Jun 8, 2016

The problem is that padawan tries to recursively add links to parent classes. And when parent is the same class it goes to an infinite recursion.

@mkusher
Copy link
Member Author

mkusher commented Jun 8, 2016

a good test case by @jludwig

<?php

class C extends C {}


class E extends F {}
class F extends E {}

class G extends H {}
class H extends I {}
class I extends G {}

@allan-simon
Copy link

maybe it was already obvious, but the workaround for symfony projects, is to simply open the file containing Symfony\Component\VarDumper\Tests\Fixtures\NotLoadableClass and commenting it , generate the index and uncommenting it

@dakira
Copy link

dakira commented Nov 12, 2016

So, this is blocking re-integration into PHPCompanion. Any chance this can be fixed? Maybe the reflection code of Symfony itself (that introduced this testfile) helps? symfony/var-dumper@f133ef9

@mkusher
Copy link
Member Author

mkusher commented Nov 13, 2016

@dakira #54 I started working on it sometime ago, but haven't got a chance to finish it

@okdewit
Copy link

okdewit commented Nov 22, 2016

Personally, I'd love an --ignore option of some kind, like ctags provides. Some way to include/exclude directories and files, or maybe a class blacklist.

@mkusher
Copy link
Member Author

mkusher commented Nov 23, 2016

@okdewit yeah, it has been discussed #20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Padawan
Doing
Development

No branches or pull requests

4 participants