Attachment 127474 Details for Bug 97449 – [PATCH 1/2] goo: add GooString::hasJustUnicodeMarker(void)

[patch] [PATCH 1/2] goo: add GooString::hasJustUnicodeMarker(void)

0001-goo-add-GooString-hasJustUnicodeMarker-void.patch (text/plain), 1.22 KB, created by Jakub Alba on 2016-10-22 18:44:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jakub Alba

Created: 2016-10-22 18:44:42 UTC

Size: 1.22 KB

patch

obsolete

>From d1c6fca2b05ed242de3becd58a0ba59bb1397d6b Mon Sep 17 00:00:00 2001
>From: Jakub Alba <jakubalba@gmail.com>
>Date: Tue, 23 Aug 2016 13:19:07 +0200
>Subject: [PATCH 1/2] goo: add GooString::hasJustUnicodeMarker(void)
>
>---
> goo/GooString.cc | 2 +-
> goo/GooString.h  | 3 ++-
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/goo/GooString.cc b/goo/GooString.cc
>index 1c0c634..f876043 100644
>--- a/goo/GooString.cc
>+++ b/goo/GooString.cc
>@@ -912,7 +912,7 @@ GBool GooString::endsWith(const char *suffix) const {
>   return strcmp(s + length - suffixLen, suffix) == 0;
> }
> 
>-GBool GooString::hasUnicodeMarker(void)
>+GBool GooString::hasUnicodeMarker(void) const
> {
>   return length > 1 && (s[0] & 0xff) == 0xfe && (s[1] & 0xff) == 0xff;
> }
>diff --git a/goo/GooString.h b/goo/GooString.h
>index a152a98..e55eec4 100644
>--- a/goo/GooString.h
>+++ b/goo/GooString.h
>@@ -160,7 +160,8 @@ public:
>   // Return true if string ends with suffix
>   GBool endsWith(const char *suffix) const;
> 
>-  GBool hasUnicodeMarker(void);
>+  GBool hasUnicodeMarker(void) const;
>+  GBool hasJustUnicodeMarker(void) const { return length == 2 && hasUnicodeMarker(); }
> 
>   // Sanitizes the string so that it does
>   // not contain any ( ) < > [ ] { } / %
>-- 
>2.10.1
>

Actions: View | Splinter Review

Attachments on bug 97449: 125971 | 125972 | 127474 | 127475

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.