bugvile, a vi-compatible text editor - Bugs: bug #35288, Issue with %{__any expression__}...

 
 

bug #35288: Issue with %{__any expression__} in $modeline-format

Submitted by:  None
Submitted on:  Wed 11 Jan 2012 06:00:23 PM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: None
Open/Closed: Open

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Wed 08 Aug 2012 12:51:39 AM UTC, comment #4:

fixed in 9.8h

Thomas Dickey <dickey>
Project Administrator
Wed 11 Jan 2012 11:53:01 PM UTC, comment #3:

thanks (sounds good).

Thomas Dickey <dickey>
Project Administrator
Wed 11 Jan 2012 10:21:56 PM UTC, comment #2:

(I am original poster) Not quite as simple; "%{&cat 'hello' ' world'}" displays 'hello world' (with the quotes), due to run_func() returning quoted result (via tb_enquote(&result) call [eval.c line 1543])...

Adam Denton <adenton>
Wed 11 Jan 2012 06:02:54 PM UTC, comment #1:

(Forgot to mention....the source file is display.c)

Anonymous
Wed 11 Jan 2012 06:00:23 PM UTC, original submission:

A problem in vile-9.8f prevents it from properly parsing %{...} in $modeline-format, unless it has the very-simple syntax of a single "token" with no arguments.

It is simple to repair so that you can say e.g.
setv $modeline-format "%{&cat 'ignorecase is ' $ignorecase}"

FIX: Change the line in special_formatter() under case L_CURL
(in vile-9.8f it's line 3608:)
[ORIG. LINE:]
vl_strncpy(temp, tokval(temp), sizeof(temp));
[CHANGE TO:]
vl_strncpy(temp, tokval(tb_values(tok)), sizeof(temp));

That's it. Now there can be full expressions between %{...}
(assuming OPT_EVAL is enabled).

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by dickey (Posted a comment)
  • -unavailable- added by adenton (Posted a comment)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 08 Aug 2012 12:51:39 AM UTCdickeyStatusNone=>Fixed

    Back to the top


    Powered by Savane 3.1-cleanup1