a proposed fix for bug 11690 concerning hhctrl.ocx
Hin-Tak Leung
hintak_leung at yahoo.co.uk
Mon Apr 7 13:09:09 CDT 2008
apologies - there was a typo in that patch - should be looking for "://"
rather than "::/" . Otherwise my questions are the same.
--- On Mon, 7/4/08, Hin-Tak Leung <htl10 at users.sourceforge.net> wrote:
> From: Hin-Tak Leung <htl10 at users.sourceforge.net>
> Subject: a proposed fix for bug 11690 concerning hhctrl.ocx
> To: jacek at codeweavers.com, truiken at gmail.com
> Cc: wine-devel at winehq.org
> Date: Monday, 7 April, 2008, 7:02 PM
> Hi Jacek and James - according to git, you two are the ones
> last working on the part
> of code which I have a possible fix for bug 11690 (see the
> bug report for somebody
> tested okay something like this), so I'd like to see
> what you think of this patch, specially (and others can
> chip in as well...) Here is what I think:
>
> NavigateToUrl() in hhctrl.ocx/help.c is only used by the
> HH_HELP_CONTEXT case in HtmpHelpW() in hhctrl.ocx/hhctrl.c
> - I think in some circumstances, it may return a full url,
> but in the case of bug 11690, it returns a
> "relative" one inside the same CHM,
> so the code in NavigateToUrl() should drop through to look
> up in chm. The top part of
> NavigateToUrl() is really wrong because it assumes a
> IWebbrowser_navigate would fail
> on a relative url and drop through, but the way I see it,
> Iwebbrower_navigate should return the status of whether
> talking to IE/gecko is successful, not whether they gets
> "404 not found".
>
> I have a few questions for you two:
> 1) is there any known case where HH_HELP_CONTEXT returns a
> fully formed URL? (i.e. if
> this change is committed, it is going to break something
> else?)
>
> 2) Since NavigateToUrl is only used once (and it needs to
> drop through to NavigateToChm() in most cases, pending your
> answer question 1), wouldn't it make sense
> to just remove it?
>
> Thanks,
> Hin-Tak
>
>
>
> ___________________________________________________________
>
> Yahoo! For Good helps you make a difference
>
> http://uk.promotions.yahoo.com/forgood/
___________________________________________________________
Yahoo! For Good helps you make a difference
http://uk.promotions.yahoo.com/forgood/
More information about the wine-devel
mailing list