I am the author of that piece of code, and I knew it was going to be a
trade-off, - false positives, or missed commands...
I found it reasonable to expect something from the preview template to
include the text "name: ", so I had to limit the match to the first
line...
I am though working on a new method that eliminates parsing altogether
- so when that lands, the problem will be fixed...
On Oct 23, 3:50 pm, Dragos <[EMAIL PROTECTED]> wrote:
> I just came across a strange behavior.
>
> 'Subscribed Feeds' list (about:ubiquity page) won't display properly all
> command
> names if reformatting their source files (eg. using the built-in Eclipse
> javascript policy).
>
> A quick code check points out the fragment below as not very tolerant in
> this respect. Quite high expectations ;- (( ...
>
> about.js#makeFeedListElement(info, label, clickMaker){
> ...
> var createCommands = /CmdUtils.CreateCommand\(\{\s*name:\s*"([^"]*)"/mg;
> ...
>
> }
>
> Thanks guys !
>
> --
>
> Dragos S. (open contact
> <http://openqx.org/?eql=created:1220736426518%20uid:60f6f10a-51ca-4ad8...>)
>
> Enabling ubiquitous content, evolving towards data commons
> Email: [EMAIL PROTECTED]
> OpenID: aqoo.myopenid.com
> Web: http://aspectweb.org
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"ubiquity-firefox" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/ubiquity-firefox?hl=en
-~----------~----~----~----~------~----~------~--~---
