Richard Heck wrote:
> Pavel, I think you introduced this in connection with 
> InsetHyperlink::viewTarget(). But it seems wrong to me. There's no 
> particular reason to think that the URL in that case will actually be of 
> type HTML, or that the viewer defined for HTML will be able to handle it. I 
> mean, it might be that the viewer is firefox and the url is 
> http://whatever.com/file.pdf, in which case you'll ultimately get the right 
> result. But the viewer might be something else and the url might be 
> anything. Right?

yes, i'm aware of it. do you propose some alternative?

seconddly for the security pov i see the only two alternatives - either to 
remove
it completely or to ask for the whole command before proceeding (with never ask
me again or so...). dont have any hard opinion about it and see your concerns
- if we remove it its just one more patch in my local patch set ;)

pavel

Reply via email to