336089 - let people find out about raw

Status: RESOLVED DUPLICATE of bug 198802

(Webtools Graveyard :: MXR, enhancement)

Description

[timeless]

experienced mozilla hackers know about raw, but normal people don't. this is fairly silly.

Comment 1

[timeless]

Attachment: expose raw to the world

Comment 2

[Dave Miller [:justdave]]

unfortunate that this one has a patch already, but the original has a lot of important discussion.

*** This bug has been marked as a duplicate of 198802 ***

Comment 3

[timeless]

note to self: there's another change in this patch which still needs to make its way into cvs.

Comment 4

[Steffen Wilberg]

> note to self: there's another change in this patch which still needs to make
> its way into cvs.
File a new bug? Or is this already part of mxr.mozilla.org? When does mxr finally land in cvs, by the way?

>Index: mozilla/webtools/lxr/lib/LXR/Common.pm
>===================================================================
>RCS file: /cvsroot/mozilla/webtools/lxr/lib/LXR/Common.pm,v
>retrieving revision 1.30
>diff -u -r1.30 mozilla/webtools/lxr/lib/LXR/Common.pm
>--- mozilla/webtools/lxr/lib/LXR/Common.pm
>+++ mozilla/webtools/lxr/lib/LXR/Common.pm
>@@ -662,8 +662,8 @@
> 
> sub thisurl {
>     my $url = $HTTP->{'this_url'};
>-
>-    $url =~ s/([\?\&\;\=])/sprintf('%%%02x',(unpack('c',$1)))/ge;
>+    $url =~ s/\?$//;
>+    $url =~ s/([\&\;\=])/sprintf('%%%02x',(unpack('c',$1)))/ge;
>     return($url);
> }
> 

Comment 5

[Steffen Wilberg]

Comment on attachment 220378 [details] [diff] [review]
expose raw to the world

Anyway, clearing review request from bug which has been marked as duplicate.

Comment 6

[timeless]

sadly this is one of the many things that landed in
changeset 1     e6f747cdb1c8