> Author: spitz
> Date: Sun Apr 27 18:20:12 2008
> New Revision: 24526
> 
> URL: http://www.lyx.org/trac/changeset/24526
> Log:
> * merge LFUN_EDIT_EXTERNAL and LFUN_EDIT_GRAPHICS to one LFUN_EDIT_INSET (bug 
> 3975)

...

>               { LFUN_EXTERNAL_EDIT, "external-edit", Noop, Edit },
>               { LFUN_GRAPHICS_EDIT, "graphics-edit", Noop, Edit },
> /*!
>  * \var lyx::FuncCode lyx::LFUN_INSET_EDIT
>  * \li Action: Edit the inset at cursor with an external application,
>  *             if one is attributed.
>  * \li Syntax: inset-edit

Juergen, are you sure this is correct syntax, i see in the code below it touches
cmd.argument()

>  * \li Origin: JSpitzm, 27 Apr 2006
>  * \endvar
>  */

...

> Modified: lyx-devel/trunk/src/insets/InsetExternal.cpp
> URL: 
> http://www.lyx.org/trac/file/lyx-devel/trunk/src/insets/InsetExternal.cpp?rev=24526
> ==============================================================================
> @@ -427,7 +427,7 @@
>  {
>       switch (cmd.action) {
>  
> -     case LFUN_EXTERNAL_EDIT: {
> +     case LFUN_INSET_EDIT: {
>               InsetExternalParams p =  params();
>               if (!cmd.argument().empty())
>                       string2params(to_utf8(cmd.argument()), buffer(), p);

> Modified: lyx-devel/trunk/src/insets/InsetGraphics.cpp
> URL: 
> http://www.lyx.org/trac/file/lyx-devel/trunk/src/insets/InsetGraphics.cpp?rev=24526
> ==============================================================================
> @@ -184,7 +184,7 @@
>  void InsetGraphics::doDispatch(Cursor & cur, FuncRequest & cmd)
>  {
>       switch (cmd.action) {
> -     case LFUN_GRAPHICS_EDIT: {
> +     case LFUN_INSET_EDIT: {
>               InsetGraphicsParams p = params();
>               if (!cmd.argument().empty())
>                       string2params(to_utf8(cmd.argument()), buffer(), p);

Reply via email to