aacid added inline comments.

INLINE COMMENTS

> document.cpp:1139
> +                                    m_scripter = new Scripter( this );
> +                                m_scripter->setEvent( event.get() );
> +                                // The value maybe changed in javascript so 
> save it first.

This is problematic since it will leave a dangling event pointer for any 
Action::Script action executed through Document::processAction that doesn't 
come from this function.

REPOSITORY
  R223 Okular

REVISION DETAIL
  https://phabricator.kde.org/D10048

To: aheinecke, #okular
Cc: aacid, michaelweghorn, ngraham

Reply via email to