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

bug с c('chromiumN')->free(); #5

Open
vGhost2000 opened this issue Jul 14, 2013 · 4 comments
Open

bug с c('chromiumN')->free(); #5

vGhost2000 opened this issue Jul 14, 2013 · 4 comments

Comments

@vGhost2000
Copy link

файл uPHPMod.pas: процедура TphpMOD._ChromiumFunctions1Execute - в uPHPMod.dfm называется chromium_free, но по коду таковой не является. В итоге c('chromiumN')->free(); не приводит абсолютно не к чему.

  1. в uPHPMod.dfm переименовать chromium_free во что нибудь более смыслу соответствующее ну там не знаю например chromium_exec2

  2. в core/main/web.php удалить строки:

    public function free(){
        chromium_free($this->self);
    }

тогда будет срабатывать родительский метод и c('chromiumN')->free(); будет отрабатывать как задумано.

@dim-s
Copy link
Owner

dim-s commented Jul 15, 2013

Не нужно ее переименовывать в chromium_exec2, она должно уничтожать объект хромиума. Из-за бага PHP4Delphi иногда после удаления функции-объекта, происходит смещение событий, приводя к таким вот досадным багам.

@vGhost2000
Copy link
Author

А зачем там отдельная процедура для уничтожения объекта хромиума, когда стандартная TObject::free() -> gui_destroy(...) из родительского класса с этой задачей отлично справляется?

@dim-s
Copy link
Owner

dim-s commented Jul 16, 2013

Не помню почему, был какой-то глюк, по-идее должен работать и gui_destroy.

@ghost
Copy link

ghost commented Feb 2, 2019

Не помню почему, был какой-то глюк, по-идее должен работать и gui_destroy.

Может потому, что TChromium - интерфейсный объект, создающийся на стороне библиотеки?
Anyway there's only one reason for that: use chromium_free + gui_safedestroy after.

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

No branches or pull requests

2 participants