Wireshark-bugs: [Wireshark-bugs] [Bug 5237] Forward slashes in URI need to be converted to backs
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5237
--- Comment #6 from Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> 2010-09-28 14:37:10 PDT ---
(In reply to comment #5)
> I started out using g_strdelimit() but thought it might be better to do the
> conversion in dnd_uri2filename() because the intent of that routine is to
> return a pathname that makes sense for and can be used on the platform being
> run. Would a user or developer ever want the capture pathname in a different
> format than what was opened? More important, how why would it make sense to
> format the pathname differently depending on whether or not the user drops the
> capture on the WS window? Seems like a bug to me. Agree?
I agree you've found a bug. My point was that this patch means we're doing the
conversion in 2 places. When the file is DnD'd, the conversion is done twice
(obviously the second time--in menus.c--does nothing since the forward slashes
are no longer there). When the file name comes in via another method (e.g.,
command line) the conversion is done only once (in menus.c). The question
(that I haven't taken the time to consider more fully) is what is the Right
Way.
--
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
