"edit" command doesn't work with "~" path

[jbhurst]

Hi,

With SlickEdit 13.0.0 on Linux, I try the command e ~/.bashrc. I get a dialog box "Path not found".

This works correctly on SE 12.0.3. I don't think the beta had this problem, but I'm not 100% sure.

Regards

John Hurst
Wellington, New Zealand

[hs2]

Maybe just your 'def_unix_expansion' setting is lost. HS2

[jbhurst]

Thanks for the suggestion, but def_unix_expansion doesn't seem to make any difference for me. Is that variable relevant on Linux?

Anyone else seeing this?

John Hurst
Wellington, New Zealand

[Clark]

This is a bug. I made a fix to edit() which made this bug rear it's head.

In the function: _unix_expansion()

Change:

Code: [Select]

      } else if ( ch=='~' && j == 1) {


To:

Code: [Select]

      } else if ( ch=='~' && (j == 1 || substr(cmdline,j-1,1)==' ')) {


[jbhurst]

Hi Clark,

Thanks for the fix. Now it works OK from the command-line.

Funny thing, it still doesn't work from macros. I have a macro that opens my .bashrc:

Code: [Select]

_command void jh_edit_bashrc() name_info(',') {
  edit('~/.bashrc', EDIT_RESTOREPOS);
}

This used to work fine. Now with SlickEdit 13.0.0 I need to change it to

Code: [Select]

_command void jh_edit_bashrc() name_info(',') {
  edit('/home/jhurst/.bashrc', EDIT_RESTOREPOS);
}

to make it work.

Odd?

John Hurst
Wellington, New Zealand

[Clark]

Ah yes....the above code is no longer supported.  Previous versions of SlickEdit would call _unix_expansion() twice.  This caused some bugs. The new code ensures that it is only called once.

[hs2]

But this should (still) work, right ?

Code: [Select]

edit(_unix_expansion ('~/.bashrc'), EDIT_RESTOREPOS);HS2

[jbhurst]

I don't get it.

Code: [Select]

_command void jh_edit_bashrc() name_info(',') {
  edit('~/.bashrc', EDIT_RESTOREPOS);
}

Are you saying this code isn't supported???

Seems strange ...

John Hurst
Wellington, New Zealand

[Clark]

In gerneral, commands like edit() MUST NOT call _unix_expansion().  edit('~/whatever') won't work and should not work.  You need to explicitly call _unix_expansion like HS2 described.

Code: [Select]

edit(_unix_expansion ('~/.bashrc'), EDIT_RESTOREPOS);

Clark