Behdad Esfahbod | 23 Jul 17:39 2014

harfbuzz: Branch 'master'

 src/Makefile.am |    1 +
 1 file changed, 1 insertion(+)

New commits:
commit d6d349d17898529ecdf8217a54987a3e9f81ce05
Author: Behdad Esfahbod <behdad@...>
Date:   Wed Jul 23 11:38:49 2014 -0400

    Give CC to gir-scanner

    From:
    https://github.com/Alexpux/MINGW-packages/blob/master/mingw-w64-harfbuzz/0001-give-cc-to-gir-scanner.all.patch

diff --git a/src/Makefile.am b/src/Makefile.am
index 5832f75..acedaa0 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
 <at>  <at>  -361,6 +361,7  <at>  <at>  if HAVE_INTROSPECTION
 INTROSPECTION_GIRS = HarfBuzz-$(HB_VERSION_MAJOR).0.gir # What does the 0 mean anyway?!
 INTROSPECTION_SCANNER_ARGS = -I$(srcdir) -n hb --identifier-prefix=hb_ --warn-all
 INTROSPECTION_COMPILER_ARGS = --includedir=$(srcdir)
+INTROSPECTION_SCANNER_ENV = CC="$(CC)"

 HarfBuzz-0.0.gir: libharfbuzz.la libharfbuzz-gobject.la
 HarfBuzz_0_0_gir_INCLUDES = GObject-2.0
Travis CI | 23 Jul 00:08 2014

Passed: behdad/harfbuzz#359 (0.9.33 - 8c1bdb4)

behdad / harfbuzz (0.9.33)
Build #359 passed.
2 minutes and 37 seconds
Behdad Esfahbod 8c1bdb4 Changeset →
  0.9.33

Would you like to test your private code?

Travis Pro could be your new best friend!

<div>
    <div>
      <table class="repository"><tr>
<td><span><a href="http://clicks.travis-ci.org/track/click.php?u=30007208&amp;id=4966ad4538d14941bc91ccaf287a6259&amp;url=https%3A%2F%2Ftravis-ci.org%2Fbehdad%2Fharfbuzz&amp;url_id=ca4c3831884d48241dd21b9f1d02f1061afded58">behdad / harfbuzz</a> (<a href="http://clicks.travis-ci.org/track/click.php?u=30007208&amp;id=4966ad4538d14941bc91ccaf287a6259&amp;url=https%3A%2F%2Fgithub.com%2Fbehdad%2Fharfbuzz%2Ftree%2F0.9.33&amp;url_id=305c3b6599d6a0254984d50ec0e2e79ce1e7f836">0.9.33</a>)</span></td>
        </tr></table>
<div class="success">
        <div class="content">
          <table>
<tr>
<td><div class="status-image"></div></td>
                <td class="build-message"><span><a href="http://clicks.travis-ci.org/track/click.php?u=30007208&amp;id=4966ad4538d14941bc91ccaf287a6259&amp;url=https%3A%2F%2Ftravis-ci.org%2Fbehdad%2Fharfbuzz%2Fbuilds%2F30600190&amp;url_id=5ca76ac97d9bcaa67cb11f4e8db47cf794849a4b">Build #359 passed.</a></span></td>
                <td align="right" class="time">
<div class="stop-watch"></div> <span>2 minutes and 37 seconds</span>
</td>
              </tr>
<tr>
<td class="profile-image"></td>
                <td class="grey">Behdad Esfahbod</td>
                <td align="right" class="grey">
<a href="http://clicks.travis-ci.org/track/click.php?u=30007208&amp;id=4966ad4538d14941bc91ccaf287a6259&amp;url=https%3A%2F%2Fgithub.com%2Fbehdad%2Fharfbuzz%2Fcommit%2F8c1bdb46facb79cfca5ebfea9a7e467b40337f47&amp;url_id=50db29b0723faf8c570d72e3c46ff7db70831a17">8c1bdb4</a> <a href="http://clicks.travis-ci.org/track/click.php?u=30007208&amp;id=4966ad4538d14941bc91ccaf287a6259&amp;url=https%3A%2F%2Fgithub.com%2Fbehdad%2Fharfbuzz%2Fcompare%2F0.9.33&amp;url_id=a276aaa11947bf68165c84ad4d1cd86cf0d0c176">Changeset &rarr;</a>
</td>
              </tr>
<tr>
<td>&nbsp;</td>
                <td class="grey" colspan="2">0.9.33</td>
              </tr>
</table>
</div>
      </div>

      <table border="0" class="footer"><tr>
<td>
            </td>
          <td class="grey">
            <a href="http://clicks.travis-ci.org/track/click.php?u=30007208&amp;id=4966ad4538d14941bc91ccaf287a6259&amp;url=http%3A%2F%2Fdocs.travis-ci.com&amp;url_id=5fa792b8f9794a936e49fb77042472d6597ff08c">Documentation</a> about Travis CI<br>
            For help please join our IRC channel <a href="irc://irc.freenode.net/travis">irc.freenode.net#travis</a>.<br>
            Choose who receives these build notification emails in your <a href="http://clicks.travis-ci.org/track/click.php?u=30007208&amp;id=4966ad4538d14941bc91ccaf287a6259&amp;url=http%3A%2F%2Fdocs.travis-ci.com%2Fuser%2Fnotifications&amp;url_id=4f5c60d0dcb18d9dd3a3031db3c2751986b22ca0">configuration file</a>.
          </td>
        </tr></table>
<div>
        <div class="section footnote">
          <p class="grey">Would you like to test your private code?</p>
          <p class="grey"><a href="http://clicks.travis-ci.org/track/click.php?u=30007208&amp;id=4966ad4538d14941bc91ccaf287a6259&amp;url=http%3A%2F%2Ftravis-ci.com%3Futm_source%3Dbuild_email_footer%26utm_campaign%3Dtravis-ci.org%26utm_medium%3Demail&amp;url_id=2000c242919fdaf375e48a3d9938df8077c6e198">Travis Pro</a> could be your new best friend!</p>
        </div>

        <div class="tiny-footer">
          <p class="grey"> 
            <span>Travis CI is powered by </span><a href="http://clicks.travis-ci.org/track/click.php?u=30007208&amp;id=4966ad4538d14941bc91ccaf287a6259&amp;url=http%3A%2F%2Fbluebox.net&amp;url_id=0fe132f64bd24688e0c90360724064f644e1ec31"></a>
          </p>
        </div>
      </div>

    </div>

</div>
Behdad Esfahbod | 23 Jul 00:05 2014

harfbuzz: Changes to 'refs/tags/0.9.33'

Tag '0.9.33' created by Behdad Esfahbod <behdad@...> at
2014-07-22 23:02 -0700

0.9.33
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)

iEYEABECAAYFAlPO35AACgkQn+4E5dNTERU3wgCdEzIWGipfuaBPERq+lyGlTCkQ
7scAoIBCQ6v8j4+mFu26CJvE9o6ZbN+w
=8gQa
-----END PGP SIGNATURE-----

Changes since 0.9.32-13:
---
 0 files changed
---
Behdad Esfahbod | 23 Jul 00:03 2014

harfbuzz: Branch 'master' - 2 commits

 NEWS                     |   10 ++++++++++
 configure.ac             |    2 +-
 test/shaping/Makefile.am |    2 +-
 3 files changed, 12 insertions(+), 2 deletions(-)

New commits:
commit 8c1bdb46facb79cfca5ebfea9a7e467b40337f47
Author: Behdad Esfahbod <behdad@...>
Date:   Tue Jul 22 17:56:43 2014 -0400

    0.9.33

diff --git a/NEWS b/NEWS
index f4fabc7..a838715 100644
--- a/NEWS
+++ b/NEWS
 <at>  <at>  -1,3 +1,13  <at>  <at> 
+Overview of changes leading to 0.9.33
+Tuesday, July 22, 2014
+=====================================
+
+- Turn off ARabic 'cswh' feature that was accidentally turned on.
+- Add HB_TAG_MAX_SIGNED.
+- Make hb_face_make_immutable() really make face immutable!
+- Windows build fixes.
+
+
 Overview of changes leading to 0.9.32
 Thursday, July 17, 2014
 =====================================
diff --git a/configure.ac b/configure.ac
index f93ceac..78e3178 100644
--- a/configure.ac
+++ b/configure.ac
 <at>  <at>  -1,6 +1,6  <at>  <at> 
 AC_PREREQ([2.64])
 AC_INIT([HarfBuzz],
-        [0.9.32],
+        [0.9.33],
         [http://bugs.freedesktop.org/enter_bug.cgi?product=harfbuzz],
         [harfbuzz],
         [http://harfbuzz.org/])
commit d218bdb26b226fbf68331eb586b24460c061313d
Author: Behdad Esfahbod <behdad@...>
Date:   Tue Jul 22 18:01:46 2014 -0400

    Fix test runner under Windows

diff --git a/test/shaping/Makefile.am b/test/shaping/Makefile.am
index 3f43aec..4cc79b8 100644
--- a/test/shaping/Makefile.am
+++ b/test/shaping/Makefile.am
 <at>  <at>  -48,7 +48,7  <at>  <at>  TEST_EXTENSIONS = \

 AM_TESTS_ENVIRONMENT = \
 	EXEEXT="$(EXEEXT)"; \
-	export EXEECT; \
+	export EXEEXT; \
 	srcdir="$(srcdir)"; \
 	export srcdir; \
 	builddir="$(builddir)"; \
Behdad Esfahbod | 22 Jul 22:26 2014

harfbuzz: Branch 'master'

 src/hb-face.cc    |    8 ++++----
 src/hb-font.cc    |    4 ++--
 src/hb-unicode.cc |    2 +-
 3 files changed, 7 insertions(+), 7 deletions(-)

New commits:
commit 3f310dc0cae9015c45ba642b9b83d5695c807aad
Author: Behdad Esfahbod <behdad@...>
Date:   Tue Jul 22 16:26:27 2014 -0400

    Disallow changing settings on immutable face

    Ouch!

diff --git a/src/hb-face.cc b/src/hb-face.cc
index 71cf49a..9348af7 100644
--- a/src/hb-face.cc
+++ b/src/hb-face.cc
 <at>  <at>  -298,7 +298,7  <at>  <at>  hb_face_get_user_data (hb_face_t          *face,
 void
 hb_face_make_immutable (hb_face_t *face)
 {
-  if (hb_object_is_inert (face))
+  if (unlikely (hb_object_is_inert (face)))
     return;

   face->immutable = true;
 <at>  <at>  -368,7 +368,7  <at>  <at>  void
 hb_face_set_index (hb_face_t    *face,
 		   unsigned int  index)
 {
-  if (hb_object_is_inert (face))
+  if (face->immutable)
     return;

   face->index = index;
 <at>  <at>  -403,7 +403,7  <at>  <at>  void
 hb_face_set_upem (hb_face_t    *face,
 		  unsigned int  upem)
 {
-  if (hb_object_is_inert (face))
+  if (face->immutable)
     return;

   face->upem = upem;
 <at>  <at>  -447,7 +447,7  <at>  <at>  void
 hb_face_set_glyph_count (hb_face_t    *face,
 			 unsigned int  glyph_count)
 {
-  if (hb_object_is_inert (face))
+  if (face->immutable)
     return;

   face->num_glyphs = glyph_count;
diff --git a/src/hb-font.cc b/src/hb-font.cc
index fc4c8eb..4364ca7 100644
--- a/src/hb-font.cc
+++ b/src/hb-font.cc
 <at>  <at>  -357,7 +357,7  <at>  <at>  hb_font_funcs_get_user_data (hb_font_funcs_t    *ffuncs,
 void
 hb_font_funcs_make_immutable (hb_font_funcs_t *ffuncs)
 {
-  if (hb_object_is_inert (ffuncs))
+  if (unlikely (hb_object_is_inert (ffuncs)))
     return;

   ffuncs->immutable = true;
 <at>  <at>  -1034,7 +1034,7  <at>  <at>  hb_font_get_user_data (hb_font_t          *font,
 void
 hb_font_make_immutable (hb_font_t *font)
 {
-  if (hb_object_is_inert (font))
+  if (unlikely (hb_object_is_inert (font)))
     return;

   font->immutable = true;
diff --git a/src/hb-unicode.cc b/src/hb-unicode.cc
index 84e5c06..d59dfb2 100644
--- a/src/hb-unicode.cc
+++ b/src/hb-unicode.cc
 <at>  <at>  -310,7 +310,7  <at>  <at>  hb_unicode_funcs_get_user_data (hb_unicode_funcs_t *ufuncs,
 void
 hb_unicode_funcs_make_immutable (hb_unicode_funcs_t *ufuncs)
 {
-  if (hb_object_is_inert (ufuncs))
+  if (unlikely (hb_object_is_inert (ufuncs)))
     return;

   ufuncs->immutable = true;
Behdad Esfahbod | 21 Jul 17:13 2014

harfbuzz: Branch 'master'

 src/hb-private.hh |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 0fc0a1022854324261fea8893678a3e9fd9443eb
Author: Behdad Esfahbod <behdad@...>
Date:   Mon Jul 21 11:12:54 2014 -0400

    [win] Fix Cygwin build

    https://github.com/behdad/harfbuzz/commit/db308280488c2ee11ba865a9922eb6a0c1abeef3#commitcomment-7077778

diff --git a/src/hb-private.hh b/src/hb-private.hh
index 68223b3..b24c9d1 100644
--- a/src/hb-private.hh
+++ b/src/hb-private.hh
 <at>  <at>  -116,7 +116,7  <at>  <at> 
 #define HB_FUNC __func__
 #endif

-#ifdef _WIN32
+#if defined(_WIN32) || defined(__CYGWIN__)
    /* We need Windows Vista for both Uniscribe backend and for
     * MemoryBarrier.  We don't support compiling on Windows XP,
     * though we run on it fine. */
harfbuzz | 20 Jul 22:51 2014

Your account has been limited until we hear from you PP-503-472-206

/* See license.txt for terms of usage */ /** reset styling **/ .firebugResetStyles { z-index: 2147483646 !important; top: 0 !important; left: 0 !important; display: block !important; border: 0 none !important; margin: 0 !important; padding: 0 !important; outline: 0 !important; min-width: 0 !important; max-width: none !important; min-height: 0 !important; max-height: none !important; position: fixed !important; -moz-transform: rotate(0deg) !important; -moz-transform-origin: 50% 50% !important; -moz-border-radius: 0 !important; -moz-box-shadow: none !important; background: transparent none !important; pointer-events: none !important; } .firebugBlockBackgroundColor { background-color: transparent !important; } .firebugResetStyles:before, .firebugResetStyles:after { content: "" !important; } /**actual styling to be modified by firebug theme**/ .firebugCanvas { display: none !important; } /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ .firebugLayoutBox { width: auto !important; position: static !important; } .firebugLayoutBoxOffset { opacity: 0.8 !important; position: fixed !important; } .firebugLayoutLine { opacity: 0.4 !important; background-color: #000000 !important; } .firebugLayoutLineLeft, .firebugLayoutLineRight { width: 1px !important; height: 100% !important; } .firebugLayoutLineTop, .firebugLayoutLineBottom { width: 100% !important; height: 1px !important; } .firebugLayoutLineTop { margin-top: -1px !important; border-top: 1px solid #999999 !important; } .firebugLayoutLineRight { border-right: 1px solid #999999 !important; } .firebugLayoutLineBottom { border-bottom: 1px solid #999999 !important; } .firebugLayoutLineLeft { margin-left: -1px !important; border-left: 1px solid #999999 !important; } /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ .firebugLayoutBoxParent { border-top: 0 none !important; border-right: 1px dashed #E00 !important; border-bottom: 1px dashed #E00 !important; border-left: 0 none !important; position: fixed !important; width: auto !important; } .firebugRuler{ position: absolute !important; } .firebugRulerH { top: -15px !important; left: 0 !important; width: 100% !important; height: 14px !important; background: url(data:image/png) repeat-x !important; border-top: 1px solid #BBBBBB !important; border-right: 1px dashed #BBBBBB !important; border-bottom: 1px solid #000000 !important } .firebugRulerV { top: 0 !important; left: -15px !important; width: 14px !important; height: 100% !important; background: url('data:image/png') repeat-y !important; border-left: 1px solid #BBBBBB !important; border-right: 1px solid #000000 !important; border-bottom: 1px dashed #BBBBBB !important } .overflowRulerX > .firebugRulerV { left: 0 !important; } .overflowRulerY > .firebugRulerH { top: 0 !important; } /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ .fbProxyElement { position: fixed !important; pointer-events: auto !important; } .style1 { font-family: "Trebuchet MS"; color: #545454; } .style2 { color: #545454; }

PayPal

This is an automated email, please do not reply  

information about your account :

Warning! Your PayPal account was limited!

As part of our security measures, we regularly check the work of the screen PayPal. We have
 requested information from you for the following reason:
ccess to your account.

Once connected, follow the steps to activate your account. We appreciate your understanding as we work to ensure security

Click here to Confirm Your Account Information.

 


Departmen review PayPal accounts

copyright 1999-2014 PayPal.All rights reserved
PayPal FSA Register Number:1388561750

PayPal Email ID PP

156930

 

<div>/* See license.txt for terms of usage */

/** reset styling **/

.firebugResetStyles {

    z-index: 2147483646 !important;

    top: 0 !important;

    left: 0 !important;

    display: block !important;

    border: 0 none !important;

    margin: 0 !important;

    padding: 0 !important;

    outline: 0 !important;

    min-width: 0 !important;

    max-width: none !important;

    min-height: 0 !important;

    max-height: none !important;

    position: fixed !important;

    -moz-transform: rotate(0deg) !important;

    -moz-transform-origin: 50% 50% !important;

    -moz-border-radius: 0 !important;

    -moz-box-shadow: none !important;

    background: transparent none !important;

    pointer-events: none !important;

}

.firebugBlockBackgroundColor {

    background-color: transparent !important;

}

.firebugResetStyles:before, .firebugResetStyles:after {

    content: "" !important;

}

/**actual styling to be modified by firebug theme**/

.firebugCanvas {

    display: none !important;

}

/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */

.firebugLayoutBox {

    width: auto !important;

    position: static !important;

}

.firebugLayoutBoxOffset {

    opacity: 0.8 !important;

    position: fixed !important;

}

.firebugLayoutLine {

    opacity: 0.4 !important;

    background-color: #000000 !important;

}

.firebugLayoutLineLeft, .firebugLayoutLineRight {

    width: 1px !important;

    height: 100% !important;

}

.firebugLayoutLineTop, .firebugLayoutLineBottom {

    width: 100% !important;

    height: 1px !important;

}

.firebugLayoutLineTop {

    margin-top: -1px !important;

    border-top: 1px solid #999999 !important;

}

.firebugLayoutLineRight {

    border-right: 1px solid #999999 !important;

}

.firebugLayoutLineBottom {

    border-bottom: 1px solid #999999 !important;

}

.firebugLayoutLineLeft {

    margin-left: -1px !important;

    border-left: 1px solid #999999 !important;

}

/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */

.firebugLayoutBoxParent {

    border-top: 0 none !important;

    border-right: 1px dashed #E00 !important;

    border-bottom: 1px dashed #E00 !important;

    border-left: 0 none !important;

    position: fixed !important;

    width: auto !important;

}

.firebugRuler{

    position: absolute !important;

}

.firebugRulerH {

    top: -15px !important;

    left: 0 !important;

    width: 100% !important;

    height: 14px !important;

    background: url(data:image/png) repeat-x !important;

    border-top: 1px solid #BBBBBB !important;

    border-right: 1px dashed #BBBBBB !important;

    border-bottom: 1px solid #000000 !important

}

.firebugRulerV {

    top: 0 !important;

    left: -15px !important;

    width: 14px !important;

    height: 100% !important;

    background: url('data:image/png') repeat-y !important;

    border-left: 1px solid #BBBBBB !important;

    border-right: 1px solid #000000 !important;

    border-bottom: 1px dashed #BBBBBB !important

}

.overflowRulerX &gt; .firebugRulerV {

    left: 0 !important;

}

.overflowRulerY &gt; .firebugRulerH {

    top: 0 !important;

}

/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */

.fbProxyElement {

    position: fixed !important;

    pointer-events: auto !important;

}
.style1 {
	font-family: "Trebuchet MS";
	color: #545454;
}
.style2 {
	color: #545454;
}

<div class="Section1">

<div align="center">

<table class="MsoNormalTable" border="0" cellspacing="0" cellpadding="0" width="611">
<tr>
<td width="104">
  <p><span lang="EN-US">Pay</span><span lang="EN-US">Pal</span><span lang="EN-US"><p></p></span></p>
  <p><span><span lang="EN-US">This is an automated
  email, please do not reply</span></span><span><span lang="EN-US">&nbsp;&nbsp;</span></span><span lang="EN-US"><p></p></span></p>
  </td>
 </tr>
<tr>
<td valign="bottom"></td>
 </tr>
<tr>
<td>
  <table class="MsoNormalTable" border="0" cellspacing="0" cellpadding="0" width="100%"><tr>
<td>
    <p class="MsoNormal"><span lang="EN-US">information about your account :</span><span lang="EN-US"><p></p></span></p>
    <p><span lang="EN-US">Warning! Your PayPal account was limited!</span><span lang="EN-US"><p></p></span></p>
    </td>
   </tr></table>
<p><span lang="EN-US">As
  part of our security measures, we regularly check the work of the screen
  PayPal. We have<br>
  &nbsp;requested information from you for the following reason<span class="GramE">:</span><br><span class="SpellE">ccess</span> to your account.<br><br></span><span class="apple-style-span"><span><span lang="EN-US"><span><span><span>Once connected, follow the steps
  to activate your account. We appreciate your understanding as we work to
  ensure security</span></span></span></span></span><span class="apple-style-span"><span lang="EN-US"><p></p></span></span></span></p>
  <span><span>
  <p class="MsoNormal"><span><span><a href="http://5.135.76.69/~paysafehome/" target="_blank"><span lang="EN-US"><span><span>Click here to Confirm Your
  Account Information.</span></span></span></a></span></span></p></span></span><span lang="EN-US"><p></p></span>
  </td>

 </tr>
</table>
</div>

<p class="MsoNormal"><span class="apple-style-span"><span lang="EN-US"><p>&nbsp;</p></span></span></p>

<div align="center">

<table class="MsoNormalTable" border="0" cellspacing="0" cellpadding="0" width="600"><tr>
<td>
  <table class="MsoNormalTable" border="0" cellspacing="0" cellpadding="0" width="100%">
<tr>
<td>
    <p class="MsoNormal"><span lang="EN-US"><br></span><span class="SpellE"><span>Departmen</span></span><span> <span class="SpellE">review</span> <span class="SpellE">PayPal</span> <span class="SpellE">accounts</span></span><span><p></p></span></p>
    </td>
   </tr>
<tr>
<td>
    <p class="MsoNormal"><span lang="EN-US">copyright 1999-2014 <span class="SpellE">PayPal.All</span>
    rights reserved<br>
    PayPal FSA Register Number:1388561750</span><span lang="EN-US"><span><p></p></span></span></p>
    </td>
   </tr>
<span>
   <tr>
<td>
<span><span>
    <p class="MsoNormal"><span class="SpellE"><span class="apple-style-span"><span>PayPal</span></span></span><span class="apple-style-span"><span> Email ID PP</span></span><span></span></p></span></span><span>156930</span><span><p></p></span>
    </td>
   </tr></span>
  </table>
</td>
 </tr></table>
</div>

<p class="MsoNormal"><span><p>&nbsp;</p></span></p>

</div>

</div>
Behdad Esfahbod | 20 Jul 07:18 2014

harfbuzz: Branch 'master'

 src/hb-common.h |   17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

New commits:
commit 1132a7dd0ecf1c425078e39e5471330bace42659
Author: Behdad Esfahbod <behdad@...>
Date:   Sun Jul 20 01:17:40 2014 -0400

    Add HB_TAG_MAX_SIGNED / _HB_SCRIPT_MAX_VALUE_SIGNED

    To make C language police happy.

diff --git a/src/hb-common.h b/src/hb-common.h
index e7ab4ae..32a62e5 100644
--- a/src/hb-common.h
+++ b/src/hb-common.h
 <at>  <at>  -95,6 +95,7  <at>  <at>  typedef uint32_t hb_tag_t;

 #define HB_TAG_NONE HB_TAG(0,0,0,0)
 #define HB_TAG_MAX HB_TAG(0xff,0xff,0xff,0xff)
+#define HB_TAG_MAX_SIGNED HB_TAG(0x7f,0xff,0xff,0xff)

 /* len=-1 means str is NUL-terminated. */
 hb_tag_t
 <at>  <at>  -295,11 +296,17  <at>  <at>  typedef enum
   /*7.0*/ HB_SCRIPT_WARANG_CITI			= HB_TAG ('W','a','r','a'),

   /* No script set. */
-  /*---*/ HB_SCRIPT_INVALID			= HB_TAG_NONE,
-
-  /* Dummy value to ensure any hb_tag_t value can be passed/stored as hb_script_t
-   * without risking undefined behavior. */
-  /*---*/ _HB_SCRIPT_MAX_VALUE			= HB_TAG_MAX /*< skip >*/
+  HB_SCRIPT_INVALID				= HB_TAG_NONE,
+
+  /* Dummy values to ensure any hb_tag_t value can be passed/stored as hb_script_t
+   * without risking undefined behavior.  Include both a signed and unsigned max,
+   * since technically enums are int, and indeed, hb_script_t ends up being signed.
+   * See this thread for technicalities:
+   *
+   *   http://lists.freedesktop.org/archives/harfbuzz/2014-March/004150.html
+   */
+  _HB_SCRIPT_MAX_VALUE				= HB_TAG_MAX, /*< skip >*/
+  _HB_SCRIPT_MAX_VALUE_SIGNED			= HB_TAG_MAX_SIGNED /*< skip >*/

 } hb_script_t;

Behdad Esfahbod | 19 Jul 23:31 2014

harfbuzz: Branch 'master'

 src/hb-common.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit df99976398a53521e0228541055dcaee8f5ba87f
Author: Behdad Esfahbod <behdad@...>
Date:   Sat Jul 19 17:31:23 2014 -0400

    [gobject] Skip _HB_SCRIPT_MAX_VALUE

    Fixes https://github.com/behdad/harfbuzz/pull/38

diff --git a/src/hb-common.h b/src/hb-common.h
index b24cbb3..e7ab4ae 100644
--- a/src/hb-common.h
+++ b/src/hb-common.h
 <at>  <at>  -299,7 +299,7  <at>  <at>  typedef enum

   /* Dummy value to ensure any hb_tag_t value can be passed/stored as hb_script_t
    * without risking undefined behavior. */
-  /*---*/ _HB_SCRIPT_MAX_VALUE			= HB_TAG_MAX
+  /*---*/ _HB_SCRIPT_MAX_VALUE			= HB_TAG_MAX /*< skip >*/

 } hb_script_t;

Behdad Esfahbod | 19 Jul 22:53 2014

harfbuzz: Branch 'master' - 3 commits

 src/hb-atomic-private.hh |    4 ----
 src/hb-mutex-private.hh  |    1 -
 src/hb-private.hh        |   15 ++++++++++++++-
 src/hb-uniscribe.cc      |    3 ---
 src/hb-uniscribe.h       |    3 ---
 5 files changed, 14 insertions(+), 12 deletions(-)

New commits:
commit f1a8d50a87edfb8147aa1bec732ed7ccbfef2877
Author: Behdad Esfahbod <behdad@...>
Date:   Sat Jul 19 16:52:32 2014 -0400

    [win] Don't define visibility attribtue under Cygwin

diff --git a/src/hb-private.hh b/src/hb-private.hh
index f5bf6c6..68223b3 100644
--- a/src/hb-private.hh
+++ b/src/hb-private.hh
 <at>  <at>  -87,7 +87,7  <at>  <at> 
 #endif

 #ifndef HB_INTERNAL
-# ifndef __MINGW32__
+# if !defined(__MINGW32__) && !defined(__CYGWIN__)
 #  define HB_INTERNAL __attribute__((__visibility__("hidden")))
 # else
 #  define HB_INTERNAL
commit 9c77027d645142794c3ff1590a96bb151c3f2e91
Author: Behdad Esfahbod <behdad@...>
Date:   Sat Jul 19 16:37:22 2014 -0400

    [win] Turn STRICT on for including windows.h

diff --git a/src/hb-private.hh b/src/hb-private.hh
index 783e73c..f5bf6c6 100644
--- a/src/hb-private.hh
+++ b/src/hb-private.hh
 <at>  <at>  -127,6 +127,7  <at>  <at> 
 #    define _WIN32_WINNT 0x0600
 #  endif
 #  define WIN32_LEAN_AND_MEAN
+#  define STRICT
 #endif

 
commit db308280488c2ee11ba865a9922eb6a0c1abeef3
Author: Behdad Esfahbod <behdad@...>
Date:   Sat Jul 19 16:32:04 2014 -0400

    [win] Consolidate windows.h include tips and tricks

diff --git a/src/hb-atomic-private.hh b/src/hb-atomic-private.hh
index 2336a9f..e6738b7 100644
--- a/src/hb-atomic-private.hh
+++ b/src/hb-atomic-private.hh
 <at>  <at>  -44,10 +44,6  <at>  <at> 

 #elif !defined(HB_NO_MT) && (defined(_WIN32) || defined(__CYGWIN__))

-#define WIN32_LEAN_AND_MEAN
-#ifndef _WIN32_WINNT
-#define _WIN32_WINNT 0x0600
-#endif
 #include <windows.h>

 /* MinGW has a convoluted history of supporting MemoryBarrier
diff --git a/src/hb-mutex-private.hh b/src/hb-mutex-private.hh
index 0fb21c2..eaa2169 100644
--- a/src/hb-mutex-private.hh
+++ b/src/hb-mutex-private.hh
 <at>  <at>  -44,7 +44,6  <at>  <at> 

 #elif !defined(HB_NO_MT) && (defined(_WIN32) || defined(__CYGWIN__))

-#define WIN32_LEAN_AND_MEAN
 #include <windows.h>
 typedef CRITICAL_SECTION hb_mutex_impl_t;
 #define HB_MUTEX_IMPL_INIT	{ NULL, 0, 0, NULL, NULL, 0 }
diff --git a/src/hb-private.hh b/src/hb-private.hh
index a451d16..783e73c 100644
--- a/src/hb-private.hh
+++ b/src/hb-private.hh
 <at>  <at>  -116,6 +116,18  <at>  <at> 
 #define HB_FUNC __func__
 #endif

+#ifdef _WIN32
+   /* We need Windows Vista for both Uniscribe backend and for
+    * MemoryBarrier.  We don't support compiling on Windows XP,
+    * though we run on it fine. */
+#  if defined(_WIN32_WINNT) && _WIN32_WINNT < 0x0600
+#    undef _WIN32_WINNT
+#  endif
+#  ifndef _WIN32_WINNT
+#    define _WIN32_WINNT 0x0600
+#  endif
+#  define WIN32_LEAN_AND_MEAN
+#endif

 
 /* Basics */
diff --git a/src/hb-uniscribe.cc b/src/hb-uniscribe.cc
index 6bdf0f5..1594948 100644
--- a/src/hb-uniscribe.cc
+++ b/src/hb-uniscribe.cc
 <at>  <at>  -24,9 +24,6  <at>  <at> 
  * Google Author(s): Behdad Esfahbod
  */

-#define _WIN32_WINNT 0x0600
-#define WIN32_LEAN_AND_MEAN
-
 #define HB_SHAPER uniscribe
 #include "hb-shaper-impl-private.hh"

diff --git a/src/hb-uniscribe.h b/src/hb-uniscribe.h
index 51887c8..001ab38 100644
--- a/src/hb-uniscribe.h
+++ b/src/hb-uniscribe.h
 <at>  <at>  -29,9 +29,6  <at>  <at> 

 #include "hb.h"

-#ifndef _WIN32_WINNT
-#define _WIN32_WINNT 0x0600
-#endif
 #include <windows.h>

 HB_BEGIN_DECLS
Behdad Esfahbod | 19 Jul 22:53 2014

Re: HarfBuzz 0.9.29: Avoid useless Cygwin warnings.

Thanks.  I fixed this, or so I think. :)

On 14-06-04 03:13 AM, Peter Breitenlohner wrote:
> Hi Behad,
> 
> when compiling harfbuzz 0.9.29 for Cygwin (32 or 64) I get quite a few
> useless warnings such as:
> 
>   SRC/harfbuzz-0.9.28/src/hb-common.cc:53:1: warning: visibility attribute not
>             supported in this configuration; ignored [-Wattributes]
> 
> I think src/hb-private.hh should define HB_INTERNAL as empty for __CYGWIN__
> as for __MINGW32__.
> 
> Regards
> Peter Breitenlohner <peb@...>
> 

--

-- 
behdad
http://behdad.org/

Gmane