superreview requested: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 162844] Updated patch for Mozilla

Jean-Marc Desperrier <jmdesp@alussinan.org> has asked neil@parkwaycc.co.uk
<neil.parkwaycc.co.uk@myrealbox.com> for superreview:
Bug 123006: Keyword substitution with %s doesn't escape characters
https://bugzilla.mozilla.org/show_bug.cgi?id=123006

Attachment 162844: Updated patch for Mozilla
https://bugzilla.mozilla.org/attachment.cgi?id=162844&action=edit
0
bugzilla
10/21/2004 12:33:46 PM
netscape.mozilla.reviewers 29156 articles. 0 followers. Follow

0 Replies
467 Views

Similar Articles

[PageSpeed] 58

Reply:

Similar Artilces:

superreview granted: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 162844] Updated patch for Mozilla
neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> has granted Jean-Marc Desperrier <jmdesp@alussinan.org>'s request for superreview: Bug 123006: Keyword substitution with %s doesn't escape characters https://bugzilla.mozilla.org/show_bug.cgi?id=123006 Attachment 162844: Updated patch for Mozilla https://bugzilla.mozilla.org/attachment.cgi?id=162844&action=edit ------- Additional Comments from neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> >+ shortcutURL = shortcutURL.match(/%[sS]/) ? shortcutURL.replace(/%s/g, encodeURICompo...

superreview requested: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 162990] nit corrected mozilla patch
Jean-Marc Desperrier <jmdesp@alussinan.org> has asked neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> for superreview: Bug 123006: Keyword substitution with %s doesn't escape characters https://bugzilla.mozilla.org/show_bug.cgi?id=123006 Attachment 162990: nit corrected mozilla patch https://bugzilla.mozilla.org/attachment.cgi?id=162990&action=edit ...

superreview denied: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 162990] nit corrected mozilla patch
neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> has denied Jean-Marc Desperrier <jmdesp@alussinan.org>'s request for superreview: Bug 123006: Keyword substitution with %s doesn't escape characters https://bugzilla.mozilla.org/show_bug.cgi?id=123006 Attachment 162990: nit corrected mozilla patch https://bugzilla.mozilla.org/attachment.cgi?id=162990&action=edit ------- Additional Comments from neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> Sorry, didn't you know that /%[sS]/ is already a RegExp? Compare [] is a new Array, {} is ...

superreview requested: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 139974] Encoding all characters for %s replacement (complete urlencoding); no encoding for %S
timeless@myrealbox.com (working) <timeless@bemail.org> has asked neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> for superreview: Bug 123006: Keyword substitution with %s doesn't escape characters https://bugzilla.mozilla.org/show_bug.cgi?id=123006 Attachment 139974: Encoding all characters for %s replacement (complete urlencoding); no encoding for %S https://bugzilla.mozilla.org/attachment.cgi?id=139974&action=edit ...

superreview requested: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 163330] corrects neil's comments
Jean-Marc Desperrier <jmdesp@alussinan.org> has asked neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> for superreview: Bug 123006: Keyword substitution with %s doesn't escape characters https://bugzilla.mozilla.org/show_bug.cgi?id=123006 Attachment 163330: corrects neil's comments https://bugzilla.mozilla.org/attachment.cgi?id=163330&action=edit ...

superreview granted: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 139974] Encoding all characters for %s replacement (complete urlencoding); no encoding for %S
neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> has granted timeless@myrealbox.com (working) <timeless@bemail.org>'s request for superreview: Bug 123006: Keyword substitution with %s doesn't escape characters https://bugzilla.mozilla.org/show_bug.cgi?id=123006 Attachment 139974: Encoding all characters for %s replacement (complete urlencoding); no encoding for %S https://bugzilla.mozilla.org/attachment.cgi?id=139974&action=edit ------- Additional Comments from neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> >+ shortcutURL =...

superreview granted: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 163330] corrects neil's comments
neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> has granted Jean-Marc Desperrier <jmdesp@alussinan.org>'s request for superreview: Bug 123006: Keyword substitution with %s doesn't escape characters https://bugzilla.mozilla.org/show_bug.cgi?id=123006 Attachment 163330: corrects neil's comments https://bugzilla.mozilla.org/attachment.cgi?id=163330&action=edit ...

superreview requested: [Bug 271959] Mozilla's About page doesn't know how to spell its own name in its license : [Attachment 167171] patch
Alex Vincent <ajvincent@gmail.com> has asked Boris Zbarsky (out of town Dec 10 -- Jan 2) <bzbarsky@mit.edu> for superreview: Bug 271959: Mozilla's About page doesn't know how to spell its own name in its license https://bugzilla.mozilla.org/show_bug.cgi?id=271959 Attachment 167171: patch https://bugzilla.mozilla.org/attachment.cgi?id=167171&action=edit ------- Additional Comments from Alex Vincent <ajvincent@gmail.com> This may or may not require strong review; it's a comment-only change, really a nitpick. ...

superreview requested: [Bug 280713] ContinueInterruptedParsing doesn't set the current context's scanner's mIncremental member correctly : [Attachment 173997] updated to review comments
Blake Kaplan <mrbkap@gmail.com> has asked Boris Zbarsky <bzbarsky@mit.edu> for superreview: Bug 280713: ContinueInterruptedParsing doesn't set the current context's scanner's mIncremental member correctly https://bugzilla.mozilla.org/show_bug.cgi?id=280713 Attachment 173997: updated to review comments https://bugzilla.mozilla.org/attachment.cgi?id=173997&action=edit ...

superreview requested: [Bug 122238] input type="image" doesn't send x/y (image doesn't exist; width and height not set) : [Attachment 139545] Same as jkeiser's patch, really...
Boris Zbarsky <bz-vacation@mit.edu> has asked Johnny Stenback <jst@mozilla.jstenback.com> for superreview: Bug 122238: input type="image" doesn't send x/y (image doesn't exist; width and height not set) http://bugzilla.mozilla.org/show_bug.cgi?id=122238 Attachment 139545: Same as jkeiser's patch, really... http://bugzilla.mozilla.org/attachment.cgi?id=139545&action=edit ...

superreview granted: [Bug 271959] Mozilla's About page doesn't know how to spell its own name in its license : [Attachment 167171] patch
Boris Zbarsky (out of town Dec 10 -- Jan 2) <bzbarsky@mit.edu> has granted Alex Vincent <ajvincent@gmail.com>'s request for superreview: Bug 271959: Mozilla's About page doesn't know how to spell its own name in its license https://bugzilla.mozilla.org/show_bug.cgi?id=271959 Attachment 167171: patch https://bugzilla.mozilla.org/attachment.cgi?id=167171&action=edit ...

superreview requested: [Bug 347731] extensions/spatialnavigation doesn't build with libxul and bsmedberg's latest landing : [Attachment 245887] Updated patch.
romaxa <romaxa@gmail.com> has asked David Baron <dbaron@mozilla.com> for superreview: Bug 347731: extensions/spatialnavigation doesn't build with libxul and bsmedberg's latest landing https://bugzilla.mozilla.org/show_bug.cgi?id=347731 Attachment 245887: Updated patch. https://bugzilla.mozilla.org/attachment.cgi?id=245887&action=edit ------- Additional Comments from romaxa <romaxa@gmail.com> This works ok ...

superreview requested: [Bug 231034] Tab doesn't scroll though address list : [Attachment 140245] updated patch with neil's comments
Scott MacGregor <mscott@mozilla.org> has asked David Bienvenu <bienvenu@nventure.com> for superreview: Bug 231034: Tab doesn't scroll though address list http://bugzilla.mozilla.org/show_bug.cgi?id=231034 Attachment 140245: updated patch with neil's comments http://bugzilla.mozilla.org/attachment.cgi?id=140245&action=edit ------- Additional Comments from Scott MacGregor <mscott@mozilla.org> moving forward the r, requesting sr. ...

superreview requested: [Bug 244288] MSAA's WindowFromAccessibleObject() doesn't work with Mozilla : [Attachment 149050] get_accParent() on document object should return ROLE_WINDOW object
Aaron Leventhal <aaronleventhal@moonset.net> has asked Henry Jia <Henry.Jia@sun.com> for superreview: Bug 244288: MSAA's WindowFromAccessibleObject() doesn't work with Mozilla http://bugzilla.mozilla.org/show_bug.cgi?id=244288 Attachment 149050: get_accParent() on document object should return ROLE_WINDOW object http://bugzilla.mozilla.org/attachment.cgi?id=149050&action=edit ...

Web resources about - superreview requested: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 162844] Updated patch for Mozilla - netscape.mozilla.reviewers

Import substitution industrialization - Wikipedia, the free encyclopedia
ISI is based on the premise that a country should attempt to reduce its foreign dependency through the local production of industrialized products. ...

Substitutions on the App Store on iTunes
Get Substitutions on the App Store. See screenshots and ratings, and read customer reviews.

Sensory substitution for the blind: a walk in the garden wearing The vOICe - YouTube
The vOICe gives blind people live detailed visual information about their environment: the live view of a head-mounted camera is scanned from ...

NRL 2016 interchange rule change: The players that will struggle and excel with reduced substitutions ...
THE NRL is planning to drop the number of interchanges and next season we&#8217;ll get the first taste, with substitutions reduced from 10 to ...

Healthy food substitutions: What cereal should I eat?
NOT every diet change has to feel like throwing your kitchen, eating habits, and sanity upside down. The following food swaps can make you instantly ...

Socceroos 1-0 Iraq: Holger Osieck books World Cup trip with perfect substitutions
Socceroos coach Holger Osieck says qualifying for the World Cup is a beautiful feeling and vindication of his methods.

Going Virtual: Technology Not Seen as Substitution for Touch
Online communication has become ubiquitous as more agencies and clients scale back on travel, but is something lost when business is done virtually? ...

Cooking Reference App Substitutions Goes 4.0 For iPad, iCloud And iOS 7
You may never find the need to find a substitute for Substitutions , seeing as the app has just received what is perhaps its biggest update yet. ...

Video: There's no substitution for BMW's E30 Mmmmm3
Filed under: Classics , Coupe , Performance , Videos , BMW , Luxury If you told someone that you had a friend with a four-cylinder BMW who "would ...

Facebook Will Reap Substitution of TV by Digital Ads, Says Cantor
Cantor Fitzgeraldā€˜s Youssef Squali today reiterated his Buy rating on shares of Facebook (FB), and raising his price target to $100 from $92, ...

Resources last updated: 1/23/2016 1:15:50 PM