user/libixion: fix 32-bit builds

1 files changed, 57 insertions, 0 deletions

diff --git a/user/libixion/32bit.patch b/user/libixion/32bit.patch
new file mode 100644
index 000000000..13d25ac4e
--- /dev/null
+++ b/user/libixion/32bit.patch
@@ -0,0 +1,57 @@
+From 869492c9f3f61aaab0d1ecdda147ea9adbdb04d5 Mon Sep 17 00:00:00 2001
+From: Kohei Yoshida <kohei.yoshida@gmail.com>
+Date: Tue, 28 Aug 2018 22:58:49 -0400
+Subject: [PATCH] Fix build issues on 32-bit platforms.
+
+The string ID's are stored in mdds's unsigned long block, but we
+have changed the underlying type of string ID's from unsigned long
+to uint64_t.  Because of this, on 32-bit systems it caused a type
+mismatch.  On 64-bit systems this was fine since unsigned long was
+the underlying type for uint64_t...
+---
+ src/libixion/model_context.cpp | 8 ++++----
+ 1 file changed, 4 insertions(+), 4 deletions(-)
+
+diff --git a/src/libixion/model_context.cpp b/src/libixion/model_context.cpp
+index 0bcf940..b44713f 100644
+--- a/src/libixion/model_context.cpp
++++ b/src/libixion/model_context.cpp
+@@ -541,7 +541,7 @@ void model_context_impl::set_string_cell(const abs_address_t& addr, const char*
+     string_id_t str_id = add_string(p, n);
+     column_store_t& col_store = sheet.at(addr.column);
+     column_store_t::iterator& pos_hint = sheet.get_pos_hint(addr.column);
+-    pos_hint = col_store.set(pos_hint, addr.row, str_id);
++    pos_hint = col_store.set(pos_hint, addr.row, static_cast<unsigned long>(str_id));
+ }
+ 
+ void model_context_impl::set_string_cell(const abs_address_t& addr, string_id_t identifier)
+@@ -549,7 +549,7 @@ void model_context_impl::set_string_cell(const abs_address_t& addr, string_id_t
+     worksheet& sheet = m_sheets.at(addr.sheet);
+     column_store_t& col_store = sheet.at(addr.column);
+     column_store_t::iterator& pos_hint = sheet.get_pos_hint(addr.column);
+-    pos_hint = col_store.set(pos_hint, addr.row, identifier);
++    pos_hint = col_store.set(pos_hint, addr.row, static_cast<unsigned long>(identifier));
+ }
+ 
+ void model_context_impl::set_formula_cell(const abs_address_t& addr, formula_tokens_t tokens)
+@@ -800,7 +800,7 @@ string_id_t model_context_impl::get_string_identifier(const abs_address_t& addr)
+     switch (col_store.get_type(addr.row))
+     {
+         case ixion::element_type_string:
+-            return col_store.get<string_id_t>(addr.row);
++            return col_store.get<unsigned long>(addr.row);
+         default:
+             ;
+     }
+@@ -813,7 +813,7 @@ string_id_t model_context_impl::get_string_identifier_nowait(const abs_address_t
+     switch (col_store.get_type(addr.row))
+     {
+         case ixion::element_type_string:
+-            return col_store.get<string_id_t>(addr.row);
++            return col_store.get<unsigned long>(addr.row);
+         case ixion::element_type_formula:
+         {
+             const formula_cell* p = col_store.get<formula_cell*>(addr.row);
+-- 
+2.18.0
+