From e8dd38d0d6c7ba3b71ad354758b4d86d288b6041 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E8=83=A1=E5=89=91=E6=B3=A2?= <940930034@qq.com>
Date: Tue, 25 Aug 2020 15:08:20 +0800
Subject: [PATCH] Fix throwing exception when duplicate shared strings occur.

---
 include/xlnt/workbook/workbook.hpp            |   9 ++------
 .../detail/implementations/workbook_impl.hpp  |   2 +-
 source/detail/serialization/xlsx_consumer.cpp |   2 +-
 source/detail/serialization/xlsx_producer.cpp |   6 ++---
 source/workbook/workbook.cpp                  |  21 ++++++------------
 tests/data/Issue494_shared_string.xlsx        | Bin 0 -> 15618 bytes
 tests/workbook/workbook_test_suite.cpp        |  10 +++++++++
 7 files changed, 24 insertions(+), 26 deletions(-)
 create mode 100755 tests/data/Issue494_shared_string.xlsx

diff --git a/include/xlnt/workbook/workbook.hpp b/include/xlnt/workbook/workbook.hpp
index 00ad3710..275363fb 100644
--- a/include/xlnt/workbook/workbook.hpp
+++ b/include/xlnt/workbook/workbook.hpp
@@ -760,11 +760,6 @@ public:
     /// </summary>
     std::size_t add_shared_string(const rich_text &shared, bool allow_duplicates = false);
 
-    /// <summary>
-    /// Returns a reference to the shared string ordered by id
-    /// </summary>
-    const std::map<std::size_t, rich_text> &shared_strings_by_id() const;
-
     /// <summary>
     /// Returns a reference to the shared string related to the specified index
     /// </summary>
@@ -774,13 +769,13 @@ public:
     /// Returns a reference to the shared strings being used by cells
     /// in this workbook.
     /// </summary>
-    std::unordered_map<rich_text, std::size_t, rich_text_hash> &shared_strings();
+    std::vector<rich_text> &shared_strings();
 
     /// <summary>
     /// Returns a reference to the shared strings being used by cells
     /// in this workbook.
     /// </summary>
-    const std::unordered_map<rich_text, std::size_t, rich_text_hash> &shared_strings() const;
+    const std::vector<rich_text> &shared_strings() const;
 
     // Thumbnail
 
diff --git a/source/detail/implementations/workbook_impl.hpp b/source/detail/implementations/workbook_impl.hpp
index 710e13e6..76308872 100644
--- a/source/detail/implementations/workbook_impl.hpp
+++ b/source/detail/implementations/workbook_impl.hpp
@@ -118,7 +118,7 @@ struct workbook_impl
 
     std::list<worksheet_impl> worksheets_;
     std::unordered_map<rich_text, std::size_t, rich_text_hash> shared_strings_ids_;
-    std::map<std::size_t, rich_text> shared_strings_values_;
+    std::vector<rich_text> shared_strings_values_;
 
     optional<stylesheet> stylesheet_;
 
diff --git a/source/detail/serialization/xlsx_consumer.cpp b/source/detail/serialization/xlsx_consumer.cpp
index 62d9c418..ede14048 100644
--- a/source/detail/serialization/xlsx_consumer.cpp
+++ b/source/detail/serialization/xlsx_consumer.cpp
@@ -2088,7 +2088,7 @@ void xlsx_consumer::read_shared_string_table()
     {
         expect_start_element(qn("spreadsheetml", "si"), xml::content::complex);
         auto rt = read_rich_text(qn("spreadsheetml", "si"));
-        target_.add_shared_string(rt);
+        target_.add_shared_string(rt, true);
         expect_end_element(qn("spreadsheetml", "si"));
     }
 
diff --git a/source/detail/serialization/xlsx_producer.cpp b/source/detail/serialization/xlsx_producer.cpp
index e6683a31..db8279af 100644
--- a/source/detail/serialization/xlsx_producer.cpp
+++ b/source/detail/serialization/xlsx_producer.cpp
@@ -944,12 +944,12 @@ void xlsx_producer::write_shared_string_table(const relationship & /*rel*/)
     }
 
     write_attribute("count", string_count);
-    write_attribute("uniqueCount", source_.shared_strings_by_id().size());
+    write_attribute("uniqueCount", source_.shared_strings().size());
 
-    for (const auto &string : source_.shared_strings_by_id())
+    for (const auto &text : source_.shared_strings())
     {
         write_start_element(xmlns, "si");
-        write_rich_text(xmlns, string.second);
+        write_rich_text(xmlns, text);
         write_end_element(xmlns, "si");
     }
 
diff --git a/source/workbook/workbook.cpp b/source/workbook/workbook.cpp
index 32217224..11c7afa0 100644
--- a/source/workbook/workbook.cpp
+++ b/source/workbook/workbook.cpp
@@ -1348,32 +1348,25 @@ const manifest &workbook::manifest() const
     return d_->manifest_;
 }
 
-const std::map<std::size_t, rich_text> &workbook::shared_strings_by_id() const
-{
-    return d_->shared_strings_values_;
-}
-
 const rich_text &workbook::shared_strings(std::size_t index) const
 {
-    auto it = d_->shared_strings_values_.find(index);
-
-    if (it != d_->shared_strings_values_.end())
+    if (index < d_->shared_strings_values_.size())
     {
-        return it->second;
+        return d_->shared_strings_values_.at(index);
     }
 
     static rich_text empty;
     return empty;
 }
 
-std::unordered_map<rich_text, std::size_t, rich_text_hash> &workbook::shared_strings()
+std::vector<rich_text> &workbook::shared_strings()
 {
-    return d_->shared_strings_ids_;
+    return d_->shared_strings_values_;
 }
 
-const std::unordered_map<rich_text, std::size_t, rich_text_hash> &workbook::shared_strings() const
+const std::vector<rich_text> &workbook::shared_strings() const
 {
-    return d_->shared_strings_ids_;
+    return d_->shared_strings_values_;
 }
 
 std::size_t workbook::add_shared_string(const rich_text &shared, bool allow_duplicates)
@@ -1392,7 +1385,7 @@ std::size_t workbook::add_shared_string(const rich_text &shared, bool allow_dupl
 
     auto sz = d_->shared_strings_ids_.size();
     d_->shared_strings_ids_[shared] = sz;
-    d_->shared_strings_values_[sz] = shared;
+    d_->shared_strings_values_.push_back(shared);
 
     return sz;
 }
diff --git a/tests/data/Issue494_shared_string.xlsx b/tests/data/Issue494_shared_string.xlsx
new file mode 100755
index 0000000000000000000000000000000000000000..87d8a17d927481cddafee66df37a19dbb4cad2eb
GIT binary patch
literal 15618
zcmeHuWmKKX((cCH2_d*caCevBA-KD{2X}XOcY*|WcXxMpcZbW&k(tSv@2-1)f9G9m
z@7KDXy}EZ*S9SGM-^4*cQ32opNB{tU4@ekYdvgN<00O}P02BZuu&RKig`K{Iou-_V
zmA<VywWGNiUN$H&StbDZz5RcW|HUIvA~*20ix#nk@B$&Ktk8N02!Fa7BA!qVnCK@-
zI*~%l00Y&@8UtMj;$e1C3z68d(~#NL!Qg@p6qMw79D1C9v}c(Qb8Bh$(NNdA3aZGU
zslHEuSP4*%osB!QyRvY*cMg|AHeoP}jLwRetdc)ux>RyxD#5EnX=qV;G1~Tyl_T-d
zIus{dVIN}51vaCJ_K$kai!&PHv)+)++anDrd8iLLOf>{(<Ov#Ygr9u#>6rGv(O?NA
zpreq5VFTDyd-W3OxImCF1@KvHNXi}259*4e39D$2CAwj1Tm7<l7ETX2PJ@P`Lj85}
zk-5nGR(A#mBO-Ut@P~(1@B;El3JUE8?{QZ+?yvM?ADGxHB3CuRS95&o`*?TvE|BSm
zarkp<z&Py}8TPq%zY>p+b+0U?tF>z$tFs@1$yiyAiV3V6d=}*7K5NV5!XctR=U`4*
zjOb%Wx!M(Z@0xpWbp%d#7rN64mb&x)T{3u!m+CGT`vWKd@b(4*`1Tj8t&yiCyna_j
ziFc!fdAC|k8+|id8tUJ^|7)%Pi%s-TT`!52l<J~|^*<AN4(Pv|S&2mC7kA(jX~LIt
z_Y_@5s0qs<!dz)1#X^+B@BtC_XmNiYTw37_JM725-DE5YK}O{utaT{$Pk6Mphae}h
ziT`3<veAw7edcE7Hc42_iR^nz1bK0NVV3y78sV3z3&9G+F=|B&D3m-be>AQnFO^<N
z<yD>gpFlJGvL~hf<#jBXhcRO*9&_>e2XH)rY!b(lNgw)cbqp7Ny7rsl-#lQ*%Nns6
zm+Pe4uoF0G>Y24)38u86ytq<*P3@N<U_`#B9~15)&b;wZsb;(wN^|OFf$A#mKOgW8
z!7W^UcVPc%B$;mrRM78lL_h!l3;-mMqZ!RV?8L#+##G1B()2gy_Ai?OdiQYeZU1*4
z?>ZyiO^fJ%=KY30yppfZgScQvLvSol;Rttd?PDGtoh1Ks2KS*!UFk3{p?x^=&W+xw
zuE}yKND6k^xd2aG25~=>Mpco*kw(J;hY!(I@o<zwjux+jx~+AwRi}VWBRl^@RPci^
zG{)|$s2Dw*ie%8)M33l>&kWYkw;_!bpP$kelh=Iy4dB~`$vT?kkWZIrf^^pt4<6>4
z@}(Vymf>jFn&pw*0v~blps`K#z}oDQKk;eQu9X;%SyMUM;EJ=QFCYull5`~?K$svt
zbko&{gvSUaJ~3$zN6jxq)W05xM+<&cU!Y$UN^X*$n%MW;!3;oubrRV<Y4eCSd>He1
zSAhT2B#5xLqWbr#cI`b#LU?cSu1SB^p%S?jixgVa){JTg-qr#wU?M(z`Iv81G92>i
za~cZptR2u=dK^&F^2RRFKcvc^R*v;=uP`|LBTJ5`i!30`j&(g=TW!O~*6RlubS3@t
znjw+6E2H>A5=l#xeD_~wek=(1%jJrQ^ul0dQ^oB&-apQ0b@kZvK1>2U`Tfb{i(o0m
zdq0l^ac&Y*wbSnrkV5<GXb;GIi|&t=i8GAXU22U(QOE?xM;z-|zhVnH@a-#3MJobA
zVkCs#ABv5u6dKyvYu^#1RLuKy)eIW=X((}VHnYEfORJHr9<K^kPjQ}s9H<Ndw>Haq
zF{b@`yD+uKTbx@_7+}tno3Y<l>vuu;QBKP3oReq{*>1!R_O8VDjbGsvXNbpB`zd3B
zHdi5y0*Lj@&uBG0#mC==ZD`*<rV;$}P#w`+)L>*3(vNK4&&j@pFaoVv0Z<=5>BxsG
zP=Zj@y0i~roK8LPqXUWy>k-W)H>oK3z?UHwGd1%hR*&g5EsF)monQH(Kwb-C7+7Dh
zl|ep!OFfpcFG4t4Myr54KUonq<g{u0sr%(C=*axdh_uHv-IH$9nJ^dTUDG|zGVwFs
zu3p|`ngkp+!Q4~GL8@FS*iuI@F$9Lg?J1r8oAA`)X$_WJR@tYOejkDB@j?rwsB_6u
zqovm-?){}{f6VwoM!~dj%d@@JZ?z&`RHsc`iE?`1f0P*qa+aWovnJ~W;zXq|ZhzgN
zv%d;Vy}@j13<eA}vIMhVuJO1%Je+SV%?04%@(4^!+UMW0JBouolF><)D;U=_=|=Y;
zTCFfqU?a>?80C%TV!&szD0l;jA1{)sH=QMapT3Z&n{yaWU$P#q;IJkr!<5X9Dj>`c
z(LU!91Kviv^{yQ>iT{jrC>H4mRr3%%DlEt!%;fD!+Haz)dh&CF=5LjJYLz0E;$6uJ
zpa1~We<-=Fk+zM#o~)gXv4x@SZ?k2$0+;0yE#eB?Jr7pB-49~r?@O7HsCd-Dc94<y
z7EP6#q6Fm8<W7~vuaD`veWDX)u#wW)rw1!8cO9FTbqCr3+(w4t6%zD7My3VT=HeAC
zPtEm+h$1S)E`}ATDB#x0BZfkdpjf5?pV_lFbd&uX1qcN;!K+;-SuKe!axAB49b?Ju
z0`(pl$13lBL@h!Wt{DA5eSiSUnQ8m7oT@1&G%nBg!T7L3<OjKb_~*!<mRyXZlON@S
z;{eIvyZS4=3$i!_rNP&TuAujW%KCJG-p|Ln0Fqd45_c>7eOxYnsy<JAa4~7@50=|w
zWDK3CkR!#54F^_x>Jg}2I$M1_()`tuj{8mMI7X`GIf4CA?0X-@WKbnWE=sRtCFsKv
zz%}_l2xB|rP%9X3cA&{$Y}+rjq9@yPwr{H3C-3c<wDebTim&Azp?A~`h3KQUs|Q%s
zG<bQ&L!gL2s)>VcXWnoQd|-t+2HT%q<p8R=y(!F5u}o^OxkeLkbSa<LksJjnBg4D?
zqPj3<=mgC2HH^#FWP5&!F{2c@-o^xY6{Y*cgPjLkI0*fmyKn1N*9GB#f-{<L-l6gI
z-pSz{JGT!)GF|(M<hfv)?Q_3v<TcUO*%4@Mfd|i_XGz!Btyf-tit5~y17#Q-FDF?L
zF_Hs}*GanVAoI(jnj6t*y}Hni6d<VtTU;LHlRArtbWTm;bxfN4p6~Z80A{+K_)F}9
zN0`5xK{?Fn#XH`WivQim68<rR+8XKW+u8opE&qN)_iv}{8_^NmNsB6U2lD#f`IuQh
zSOHlUf_nKq5S_<G@TG*vJJc7KDlGl>ogJ&O9p;oL&L5Zt@KKc0jlh1}x*v_o&s`R&
z+W0D150_D)s5m3Fr>I+#LB4(!3+jth`T$emCr$K%-sRrCP!SIGBa(Y}K{UvTcHkid
zbG)R;#QahdqMGJw%^Bsko1zapmyGi3akv{Ur$tWNsL17*uP9rj7I(G?TfflBrKb+x
zk;84esT(|KzWMOb)$O|h{ihExFv~3r1_uDpFaZGc{}_Qk`ktDkY@5T1*tb?^(gxDw
zVpfjQC)$&gJWqEwJ!=qQ=)IGOQLcoyH|u_9jtxGMHb5u)N%zgjUfiR`mx#KRGtTd(
z{g7nu$?qf_Yf0xR)bjFWrStVF;$~vGHko<Fa?<;3q0`+U>}BV2PLmqP>7`|XZdPSo
zdyVo~eMw<&%R1&nDT$d-qk?OB;j&Re^<H@`ETzcc;0qdg$VIoc!500<UG1fnhDSHV
zn#EOUspEIeO1I+F=@JpuLXWDU;@$PqVeZYJ5=UouGt#|7&}z%OLaW^31-0@9dmSt3
za|h-?C+;>8Q^U?h8<xOt1p&0tF(UfuGtC=I-$B#+ibks+47aMRlkL-uo_Mx<xp3x=
z@jqhEHzhx-#VyB-&gD8Vzla)EjV47FbGMDLz&vRRzj1gMQ*fugZdsl2JYDkd4p`QJ
z4fMC>5np$NiLlJ!Xlj_*?7L0q6dNr0z07b)EjQHT!@R^*_4VFc)Eu0^Q7FyQ7kZki
z(xfJ^D65oP5^%t1!Ed+p@2ifsg=_+$vu7h$CPKy<n7_8imjJFPK4MHi6Y7SXAv!J#
zF}LBGRK({1Fl<wp%?3i+t|XgKw9QSsY7g8dZqG|}+pdH&b{v|}D5ZoVLqDz~ahKA6
z%=r+_=oUREd~$APN6>pzC8B&EzrsRAgD_DIBai;5h~{5R?F#&AC;gH9;)}9T3nh%A
z`uD@o-Kii3dr~*o^Jo7E-4l?Xi+$an8me>+4eG5nPBFRh;!7aZ5P&6&xK^#{r@pQ`
z7mBR;I}D~DR$9&Ik8xw;k|-@e)|=q=Wy}CMi#K{6nFgJIB@{#};zai0B}Inx==ZC8
zYCowg9|JEG)>PUl9i0~_YpObEK6_YoC+LeFv}MayFiz3Xn0}t{K9D9diVF@1bAj)I
z_xH}!1Oo?c0Vm7@dbT=9R}00}A~IOq6<bOq5eSK^O&q}BZKo#RzA%*r>y=hVm0v=a
z-Xklj&6Mtey%JM>e^&5*N4Mnc-)Vw^p>W-VqyT#Ks5zpOjR`*pV=9LUA^#<bxA~_f
z34`1xeS$s9g{(RB61sERb2>;`NV-7UKspgvT4o0RVFcWdqDq8Ok-&<B-4a^M+Jl|r
zkUmV{IN;piK;Zb`NZ@qfK|tUb9d?gUVZKb0M&8JxNwDB-gEevVbb_5)VO{}V5qzF}
zfqcGvp?o@?ULxnq_r4;X7(u=Pt@LbBw(H#;RqJ9a%%FeMmRINQ;v$NqB#O6yEF^Y7
z0@}_u|0u8VN4(1BK;<O7i;ZM|GGYXxW|YPqzz5L>(g)qA8vruWJ$3CiLRnG5d9ZtH
zk+m>#(h282iABg1H=<=DjIf$Ve~{8T=?*6erHrzPw5i+or+y>{Gfk2rOD7Xv^al&p
ztIdO4ocrd#po7FmGU4OkbK?Wy<KrXY)8Pk+`UhwAEOJmlRfG~9_~QaCidfWdi)rTt
zeU7x5j$x{}fV_Z$f`oz!f((KZm7qt3ET!<ki^M7@Z#Y3~Dl6r<pxh%T?B8{t@gwCh
zXXxi)Y*k$Bja77F-s$Nk%yZ=j=I7@p;Md_tj)C0~g4wUzPLt}cRW@fl5F}%HGNRoI
zI@bcxZB)d{I}^i9GAXjJ|3N~6^LYma84(#783`FBI#I81x9=8=URjFNt+KU}&uRi~
zU(8HOdmkbCL#e64`;Ud2AqR#D2TF<Xiin5^ib#m)L}%IIE>RWkUDy$Q<#A68GI^h{
z4mMkS>nRLmoW7A54iF1bgh}uXgnQ3B`riA3`uh3``gZ#gk$i%FY$)|xI-%+o^;Mcx
zH?|`{=-%d+LXm~9biiZ&W#DX@D}xZLvc(7*4FhrnlW^Ts%g9u4A|Fa-z+CJkOJ`A?
zP#;jiQ2kJaP<v2`K^WzZ;;?&COb&XQf1vS+J93LlO4dL&<|~*6a|XKx0|)a569nr7
zcTYHJy5z;Pk17OM&cLreS380C&8fCp_Ds|(F>)$e^)~N`x6YMCbQ={-a>VD}sJB{{
zc$;r1?}y@<o7e|awbeC8qvofmu6ABrZ_L4Kz|*0pk3ue6vRNYVRIg#qt|}_TDU`Gh
zXyUPQ(mAGenc_X%8fq&xTO2W3nwvsAgB>QL6_W=vSiz6!7ZNje65TX7;!8tdvdM=x
zv8W`xXPf@Uf2|i9HzPp5r<@c40JwjKhjvb8`oG78(`u@gTfHCLX~y4p5%+WA9Yh%^
zA>z$|fhGysExcMMQHiTIgfa_DQ}H|}477=Op+Wg*cvTtA%U8JV>WrSog(l73Bu;AH
zgvMYk8IXy?^ki}sl~i-t3|raOl59tZ@ty8Rmu(vgzO&i`xAlof2&PxLQJ}NG*7v>L
z2TdGaE;G`<WF3IFu58g9e|{UCxNCD@4Wm|jx@a&L#tJ+y;Ckf@CV}v#tr;~ROXY?p
zsYUSk1eRs^#R@&#?|`U!WCNAyQjHI-r*=Ile5?-{o-;%!(9j=un1qVroWx)|mF*{r
zx|TBMo-)K)_zNE6n3^Yhg51>Rt_(+PgHz&n?=k9|Ej9&o2_NZe$ZhBjqiv;IdNuWN
zu&F^}uk95g(PX^A>G{2l@Mt;w>J0Wk?$>=Q)cc9pD(@uR8%JfyQ7r7QJlpBX)aMiz
zofBCf>iPPGcu|a?GA<;?!pj_=E~o0&o<FKH(R1~+1da{vgOV^e4BPUK?B{wmhH_B<
zz&u)pEzpvu(-Lx{RX>6}Adw_b^n8M=Xc8odZ=7ekOK0GYfc~*usC!iacAbt-C_*Kw
z3sRwa=Qk^4k9!owxXq6(H}i7IS=9KH{TVBK%(+?H6E{vUs^{^XliJdo_~i%$32%DA
zL$%oUIXDYFt*B2*nMd0l3IftcM{c+S9JbebFZV0X?c^IdHql0PuW=w2-_D^!KYF>D
zKQ@S)34K>7L-M3-*h^&w#PbnLnMK>v)5=-;irW*IZT}LTeUpbKX8t(;7nPBD<T_<W
z!_z7J*#p!|?LPY|K{e@b1n`F8@Y(~&3t5)!KnboMHgsN$%K<YVbiM|bEm_^LGSU!z
zD$p{6%lA5hMGC$E>`4rqL!8L!S_MB5M^<Dj;ZM_or$PIs-`?;&h(NG^UGhT?pus%<
z{OO>=pF0l$w~$=2An2~=sD^$zX<&;0g37Q*7+Za=Nv3J(<gkd~2f4zfvd%;}Pxany
zd0(Bl>=ZpP9OAy1qU8`kP_C)IxP{;wfLC~`F*$dL{JC$m(iv3$NN(%MAddGNN-?N+
zvQm#}Jo27N({}|ZP^JnoyuzTFugcm47anPg%EXA%-Q-@GBnlV#nnAW7Zf{J(e;4`f
zxry<BA-&?wzZ$KtmTb;^DA^nJwt^I@tga>qDvu8y!^UpBCG(C`4g9{e;eaZ!zNd^R
z^i7nV9gBUJ^<45}>B%G-ELGM%_)QHCZW_ld#P}!qsvz6r`j8^${$`)xN{gDDU~KuA
zrHjt|+@D-7`@_vU6Z&o+FCk0>8&^23_yY=A(TZu6A9gY2fj4QgA}YZ0F&IYX0cHeD
zP`zmSp>%5~2H1oX-sYi#AiZ?RRAQCc+O$5-X9OMo6CZ?{*%uczg28^y?e;PcJuc7Z
z^5$SJ#cH#-eR8ux8(?J#L^Jy?z^GM?ncZXYU6%EtP%>oXF`rr!g3$T~k&SZ69+yz_
zHAWhZTTU)1BZAdq_mjpowV6u0`NzV@ba%g#j%`yc@=Xv({9=qB<E`~bhF?iedm;>0
zldv)FHDH%9xkGC4uX4D<tsgEbjzZnWdR=H-DR*CJ1I;Kp?}9TCjq(#RRYeH;`0vn_
z#yi8$;1VMg2>oX)@pV2Q1cO=EKxz{!27a@E<yVSBC6L32Ks;{A`VM94Emy$H$o4Xe
zzk0Up1<TR{DaV3BnA`tJ<oqif7}P4s3=wNC77?C?ZpO?bHq<dS7#KGh1Q_HI2$;VD
zWRZeoDYQ!_v%#GVU6Nobh(lTo6vRzb19>&m9F=&nj=B#%N*3t1-D#gEi(q@?D9~2s
zDN&SDM6kGBAK%Q$aVz2Ru<T(F-i7&R|LVcsT^1q{@;2o&rn{68v0Ql>C@EcOnW$z_
zGSN=_XT~)}X6D04h0iR^$KS{>F&!X_V=*zmi$sp7%cHa+r=K|ikg!f=%vv#{e|ubP
zW;UJG{7ttD-tQ8><a=yD`JyyVw~4S5q_&~gsY{$9WE-EP2&+=v9e}CO?Sp4v>6|(^
zb);WS{qPsLc|6^vyw2VcVN7vzvQKLArs64m?9AoNQ{HG5V`S2KRaba++`i({7qBy`
zPT7gz#G+~e>WZR1E}j(NzNhLxfcexsN;LSo^_1*>?9eL2c^6fE(&1>sh+`ZLtfP_6
zsTrhZ;9B8cSS`ooo~Y08+uaW03P49%Q<g1LRWYz*;Qc_A8O=d8k4@^vz&}Wl<FWQQ
z8+eHQs1dmAhQWxn+N-SM)VvU5o3~}_XJIbvcLKK-{%MxWfcKQ~8}z(dUlTI+=omgB
z>8WSVna9wj13XtBY*{gqM|(|uokxB1%XJ188RzIQQMGdt8#QOKC%tVoQLTY&{YxEg
zh6KTcinf+|1#R<#2&4~Ob*UfUW-O$D%NJi-Kw}Lp01>83{6!6&bFsdKcHT(bH!F<q
zJ+ZZd@u^zWcLLNX6tu9%EZh7ezacLJ4854rD9K_Tv2VwH9a?-<5c3?QHo(qQn=Qpi
z$`=pdyrA;idtPE+pz%__9ka>b#H(Qzy7iIx4MV?(Lk%tDhAC(>s{96HUQM1eA=7EY
zJh6(z2gsSr^nDl55YWHIh%FD5WTXa8>J@1pQB-vBcs1YE{`?7fi!xY$+dX+Us=Hr<
zfS3W;`Ad$AyOIP~RCI-uDsnKMs>py*K>_w6r$Lm>%bDU29peZB2Ke2_o1>+V8M*XE
z-WNwgXaefXo<nwp+4G-phOr<MaI;uCGd-y8%d&9qRi6bWx7yr7zb7h)jMxXa(us0u
zkls8v9Q5xyc|y}l5WQ?2?p`hp+kNz<T|7N0=4o-eNs)j&DUl9u_uW1kO5x#ddwo`%
z8*{h5n(?4Fu7(;4{rPs=Gv}a~v7**m|8O%ia`qN4niKX-H}3Y5W=_+!7SBXmf!6u8
ziDqs2xl$|f+EqAD1TAkx{q@<SOfx?xC#`yzRmyVB)yeHfG9TaBKQoU329aoc;Ytn?
z8&z6KH>x7Gh+4w&OGH%Q00_Q$#Q9~S`S><|Br~?(#0!xTJq_7ZBj2b#%*wWQL^>Mz
zmW1T^_5n<(PIXcjqvG3*Vxqclh~S`swSi5Bm4Q<a5E@gL(<>{&_aP+b12Xq<Qq^5g
zY`BvsHrpK11zZ=`*O!3%3*q)?wVT!}#=!DGoD7dXOKV?G=&u;xS65RytAfu{QvBfW
z@21>^dDxh?6%>yhREu;>snm}$su}IadD!Ueq|qR7yR;@SRS?!^3ho>)KD*0Opvn{7
zXdQMQNq6a$L*4g1G~h^Kb)Q=!foR3d+c!|*m`2ajqe`3nFt1uVto=}`Zxro1sqj>g
zoP{)BgRL1!4jql!X7a&n4xXT|O1)32LF^O37_g{Sy#XTspl@Uw_}%{7dtCURnUA3&
zl7QUz%ts8yZxn;yv*&h3`sVsHzkmOp@Htji4Z&nVb*H)JfxSAtXWkk@wk+@aK54)L
zrA$g3QAy<zI}<uAvKj{E70DULpA{M3j4G!2_)QE1j<Fiyg=Qr>GM^_~IJrVtWoUtD
zCAM16LD_&YBb+<U_WI!a?Q&@#I|S>7Zx2<SYyLJnD^3MnPyyB5dn(ze9{wX44&k8A
zvdzcDroCF8=XJ_OWc=`QkQ?1rKNSnIZnI*gZ9=jy&#);Q=&h1U$oN2I1J$fw$MNUG
zj4EWvXM&LkDvr9zfU6QnQRq8Iai!edUQfE#&fLL0PH!v2aQysrI`lgNNn9$y%;e<e
zFWi+0?!6%xt>F>P8^)ItvEk3DHrP?EfpAqUL|^KAPO5~e<K(k-1k_74G+*>?DxtSt
zf=w8b;1$E1)6ZrWDW0Htd)R*}p3OMd#BqV~<{(`&?qn}rbBuhQ8!N{kbE}>|oo$+_
z^qM<XoMWwXdoW%9Kvr{By7BY^Xf}Gk?s-DXn5OVD{OxjX_WVYs>1YP78Q3qj`^j`a
zTC*5@X!R!^$`=}?4av2mIf|qUEU<<0LF$EW-yR*=pF;a2;*Wqx-eAxm?25z<C=5P&
zh`?xV{#0ywIQ$C(G6Sq&@i>__Ils5dIa#SeLL(T$(%Nk>iF=i(?X7cZeuL~CcUT2=
zxXbH-m>idgh|?EWySLYg7Tl`ZxX(GOV5m|z&NQu0n`LEZ>>jVzBOwr)`I5MOw%BjS
z0a6(+mrFy#0ui=~oG*tHRecfZ+|SPscL<cTOHdex=RiTU4%8b0d*mURK4`CwiOm{e
ze1U<|Abwg)I*p|KAdbdfg>Z8n`7mXqne@!-4-b+)M;eL4U=`J_bfK{8)VkGO$i*Pk
zgSCtZ2)h^4tB&+@){02%<u%xu^N?1S+#pWG%rSg{aLI~f!C=Xo=rF?Wx=-k;&JI3^
zU1d#rv9zNkoeg}c@B|78u>IpAxy_Sc$b4#O%-)D5T`Wy=Y4AC<8V(m;LB85vg?`Tf
z*{-m?3AN0A0~=3*A`VxGpz$v@mQkU0Y7sDj*Qm0)*S(3hs%b+|*NQNYLf_C`<{b_Q
z=%a%0o|scO0_m_9^q=?wvy<W&f&{Rikk$fFg+@P+n4PN|gd;w{Tk~G@T4|EbL*5PW
zTfue97sQeVKGRE4R&YinvEh8Q^DdY9H0~}iNr;?Lc-9xY^5zh`@^hjf^Wb~eo8ZV2
zR7Lm?m_9~VlvAF<Wp&1gGA(q<BZfJb4R@O5(>INJmav_vgd?>qzUHf%MwJF(%#Wui
z{0Heu-<rE-7Sp<#8BYbW7Uc@lc1#FaU^eKMNYLwzz2&e;wKq`+I$ID&w7fJGmc~Di
zNk}y?=wi*j6e*pAT+l6<+|fpSUtnD+iyyMFOb&(+CZM5tD7Kk54MbHh-g7CBbE+v$
zD{8rn_AGf~C#vq1K$eFIdaOjuKBq@6GA!uKHrAeq+)c<*`C`2Tomm?UO__?NWKpBG
zCO!tFCGJOtGsK!F3_P?yxSnc~%fc3O_z6%Ps1Lp!Wg+$xvx(FtA%BrVa1k#nFGZAJ
zybCeRIzH{OW5Bq#F)3g=lT^I|3yv+!dCR<8CAgo?JhS2_D&u!}Rg2QYE~Ev>5tUQW
z+n00n5j=-_wQ~Y?KZfckkx8I8l0ka$`h5bNe0as|m?Azw(63lx+3^%jZG+$#6mZhZ
zgC7um9pcqc^mfz<A^2E)LP=3Lk)7vjkPi;7KcI5YKp!c}zl?NLlB5XK-*sMOG2~ut
zlY66H9qAvm(`{Hi%YJQPBe6<vK(6F`3zw8gIzEI5CJ&gj5j4afG(c4!maRvg_<(&=
zd+Bu>fAa~e@w9V8?rZs*<1Cj`QGEK8AVUTNoP1POzPiNPDp6U*AgQ#EJo$>rQoC)b
zDRm7z@(jCpD(z8gr*Ee85GF&RXzvj{B@+h)(~q=GZ{90GbDuQ@Ti+QJOZF>)mMvup
z(!iZY8RFK)xA8NQ`E!y$;g4Uv_T;QSh}AoZAW}X~+HT7+YXvSNFJK_0r#kVgoSsCl
z-FgS(qKd(_DV`T;{1`u9n&Dr%E8a=hq_*dX<Djhwq$PhkSP{pC&)z3!@{-;$t)rc|
zEIAnt6cWm}P!0~RZ}j`DXjVrH;?%au4l20!y>!x2Atg;EOWFJmMx$<ujL}D>V7wIU
z#-a`LKFr`P{0NJzeC{W>M(G^c^dR52a}#3Gwh7AV7O}jOnWeDwxz-pe-ORlou6nJ2
zJ93RBkH1ZaY(fmR|9s!fz<Ym-_)ad+v()`&V`*heqitpNPtx3fDFyG&J~g_-WcfEr
zfnTf71Q&&+48O9TU%BW&Ru)XtfE#(lTy`x{ZqUNX54Q;Z1%a$OEEvDZ1kp+EFZvt{
zjw$KRDM=ppds9Z;6GcCx!QIAZ^~^ZMl)b+!sHN!lQlpjKYE%*-CSsAB((0PR9*b~$
zkQl78U+2;hO?3K(#tB+<I(;LaMf@VoEd9Bp%uDAmVHwR&!BJ;MKFt5#JsHPF4JjGx
zimQ%EuVpJY>y2ro)a0RaN9Tl~9JOy$UInN}anQE38Ll>S8!MJsParAvC#r%s!jj;2
zYEI*2@+|!a*TmokZ|DMRU&>5qvU^>=RQ&3r6t7#pV1E)g@=9)~M>n-~Zt+$Ab~H*|
zI7c{t0<EQ4QgyWkUX4}vC!4FSsk0Hqj-1uGMyq-D&>I-GQw*K*rFo2NEa`b-HKQWh
z=FbM}2e|E9_*Q-6TdRcbudk+88@qeEr&~8DgQqLjtC}eJwcoX7rRTqt)i`~mUhYf>
zY;=8f2+`5-?&Eat`o6d1%^GraEpJLs)j>(;vF}7)U1zm`+l+-9KIy~gews;gzhykc
z^S6Q9zNtpjZyR>f@4LcCf7WqbOB?<FD*5}E{qt~*mKN`#MfF<)dlMLNNvXEi^{)kq
z<6=WBA#3&9{z<`XERwzQ+G5-^1MeSkmmYJL?jl;t-A{54q<f^WLlzDR&3mCRT{JkM
z+5n|URv!MjBUynjJX~v{vA$)XKEIjiTt@}2cSZ|DN+#YWEvLmcS;y=Fho~ybfsE!<
zg)r$V??<`8bBMer3V*zkx2`vvzl&BODFHh^SVrXI5+ew&6)c>{x6w`3;ERkouc+HY
zAHF0u{gd#;GrM38Lm2Mhm?y2Hfq8rN={y1044z)xrM)@rnR11fQr1iYlvEp#;hOyZ
z^XBsa(?ok!k1ch$&5ivV3a*ZTdO}dab&zqoQ)MS9+>Ajua%5y0{z}+^>`+C<)`Ei-
zL!Qe`H(VpL{;743ASkE-atSnI<w>06(LlrY#HULX)VB8#{Ld9b%3b;-?K?l&`F**7
z0w4qE+S}S$nk$)`(HPt5oBzIIz{@5AkY(ci>mT?Z&&P?%wit?terHnFJMtgIA*mv5
zj&(jh5innnRFv^k0$*&!xE4S#l0~63r8ei~`zJMs-yKZ3J>Blyt#LQdSRCbBY;xBd
z&e_A%*$Le38=NZM#lX}@oQCc}I2-(wbO~5ZI#NAbEL9tO87`#gb+yh|A^qrZSH0TA
zc{d5Syf~6{Vw^ZZ@B`#aiP2k6nhjMH2k+~&f$KI~uD;h#Vuy$*T*<50iZjtSsFtWX
z3h3H;HzD<fl}0!bWNKq7x~G#XRa<*K1E=_?H9Ps^jO-J<y=@VV<U7n$G8^3Gh9<^b
zxLqCw9=~Ix{xPRfHJGr3Q*UcH#=9uibr$dZ65|_d_qC<=l?Fu0-l)VcHqa@j6Po@7
zepzQ-doeCqT%f_=3M`2XU^kFYyYqAOjGq%Wl046_u$J*Iz=rL=2^*K<7YiTNaBQ@+
zGX%ynkFni{IJOPV!#hB)0vpYH7xXfO$w}GvoCGFOK2FQ(eL)(7GfGNqjSuOVMlS`?
zBb{&#&j(_UXC~~aXMSuF!S=aaM3ftOl4qo;Xw*T%wAOwknNJhi3T{Quu{eO!z4{CZ
ztkp=!ex_B-e+HM}DBdwv<75=}DokK8k_Z!$%84U>z6#tE+fXnp`)rUn6_7?H(d=pJ
zH48uKJq({ow<pHg7n*M4piMt~j|7>09tm(rWf97AEi3${N8FGcnn9cGyUbMevA6ak
zz7P?PPM(Q${YITv3)r!v=K!>Ai{Th2d&f^SdYH_iiL#knPk(e)S&}295Cqm%9)g^H
zOfQolO_pY%^*rE@_=#uP51^y*N`z0)c%@ksMzy=>34I{ik*L}hpb5H>@UUy(qbR#i
z+-s70c^g?kTe7Q7)hG99O0Kn$)KZ+C2XGgJxWngK$P}i-ratRlNb2RaK@p!NU?*0E
zdLfg59!<1HRF-S0vSZelZOq%5L7kApRYl71iw&Qds32Ci3fL{3C}-|nXi@J^@rk|i
zP3=C*-fA@k3gbr7^_H1WCFxC@wS6Cpq*M#Oq?`mECYx#l+kj2tD508S$Rmb0Wz?mg
zF9+g_RK{Lapv5;JV&b!eW;Th>iy0PM806NMz&^6!WDzM*MkRZUManssDR8sUK;2`>
z3|kKqFq6|*Lk<nm5(QPKQcaNk2Em)jSuY>}SVrMQ5?oxq-0{oG=-h2fK&V-znTYk_
zbJ7V|<$NZa)zyN^Jc@insopl5aa%b>XJB_M_A;!dx9^ES7iq3_UvmDu<wr|dODQ!H
znn1RHYNUB#mXt7iFQb(3^o%HUgg6xYA$RZP<uEp|9(eDOj6v<BD!4hx<7RfQ!e6tG
zsKa)C4)qMqkxTam?0k(lT><pu{`Ris#9i|RO{1pul`Z`>eTwx0bz!NX8Ml^craEmM
zUe9SMVC(^BL*N8wFocl6Ze%LNrKfWRQv<cH2nCfX-M!*$X9omF--Je0v#MzB7?gO|
zH{$jV-tlnVcx%jOmBBSJpbTbKEcvc*XYa?(QT&Fino$Incb&0|a`;O2m_yJl4E>Ko
z=mB|bx~!1|9s{P42YBz2Hw>P^H>d3Ep+zdhkHpue8E5C}3x?H*&31_es!#k&v&DUe
zFzWcH&DPmwD0pHpnPQj4k6#=WtQ<omHK#b<q;9||GstMty#%b;E1kuaD}3wZ1vs@_
zur&)kOmBkj)2zmfco<q4T;FOeWU<oboh4O-iYPI>Ax^4zs1@&3c3FDFts*R;s28%D
zGqK7rs4R}k%pFPRKM=0UoUe6fpqv+vPOrxEhMUBLpNOm0>2L9E4b{FGmcT1wm+QKB
zT}sbvN;L_k8X33t*0llGtT|EOe{C)adO}RNO%cFG0`oZx8IDrxQFU$;{~VA<rQs{(
zWM%e9Q!x-CzO#txMnP{8TA`v?1a?`g>F8?8;+{ftq(sp2VJKtK5*1>xr)DhI@yUa0
z|Jhmz0i{?|?&$dT*&x)RR5!x%P1PEy7h<sBwe_TwLi20b^JKNFLT5n$t7{qj`Q~gw
zS&YA8f&x|BN;1w&>EIYh*G!*BgWKy=<JHZhwRP%R3+E5^E=`_QXZET=H=UZ?1gV$r
zX`GKnHy30GbCeM@Yx5q)7o{(HKFuG8T<m)XPQzzvv=?I|s+*>b?$V(WuV=d|xQarW
zn|DeT(|G6NGZ?zdHdP+>#+nDDFlV#I<}T`S)oDnzxaBzbGZ<i2Ph-|(&Lko(1M4)u
zlr#GaF;GrF2+LanONFhg7aYTBvMWlKJ2i~4OfnX@>62DLJ`fv`UQ<fX^??W&1Xj^c
z;%FRUw|%{gB7eg^mdShk9{QD^bX^HgyisaHBdO2`=UXXvfs98lu#f^+Ms8$+K=x>5
zT}c5LeTsq$_rif>P19jem%OVimGh0C;*gVwMr0|9A=*jAE(l#tl^txl91R#>WR_td
zxBL>trlkBmFe6QQ=F|XJp??X<39kO4>&skdlsdb1-f>0%h0X)22F<ub7;0%UYlBnp
zv+4B;Qi^!>SbYdy@kSCg!&uVCf>H<;_JX*5C|VsaKSD%{NLql`YtlYlW544JG4W40
zE>)|v_E{+UE2UXs#ZRzfa%ddQg;h3*#XZARkksu@dM;BRs&LAy#)iS%ih>aoVTU=c
z<@s#8?hR+PVZ$}d5I*^QyRiq~Jq(?ELw2aPi*ke^9b~)Q0S&?E9Y;l7OewmiX;0Sc
zPA!8ZBoV>(<N~e5+A7{J!lTl|X%JKajF^U)&p5LvU33=$v*SMS9KXDka5w#Z1nhCG
zcq;TBTP(kSNZ&v2#9ym-GNhfpjjX<%-EXYOU+?Ltb&M@^v@&67D`~4%fG|2r#`@d)
z+XrYvmV61p`Zo;&wRC_dktBwBC!;!i`CTGw82H)nlA!|mu08Oq?qI)Eyk|=PCM2@s
zVRP<$e?#^jkK??@;(vrh|KPO$J23h$q5XZ7{&o0{bp!*`!Gf<ht#iaKr;R!yMjbdR
zqcsQBuh-boGxQkCCX+qhqDfIu9@9ZRyymX(be5bZF32*D%Q2CKhCAa5gYfK~&7h%%
zgter7&h{duKW^H|=GrjJqrBKgDb5yTrVYwXn>{9hji(SpAJWUjDv70!U{`jiAw0BK
zx3qgT=e^S|60pR>eFUF=ND{kzV}|o_xEDG_qaDOWT3k^gQ|ojcj;-%!jvxbX)^e;%
zE?S0ZG`;EbSovETWuWx+`rdoz*x$7P{XYWZ-xCo3U(vB%#J>suS;JhCD?g1Lj2jEh
zwGlJ~J4{7l%rggKi9YZ>pXEEMaKVkiUb!<CNO2hA4np{Zcd2Xe=BIW~)WZbS&3R2n
z`jIV@8Q2B;2(Z@Fv5cUH=L8Z6ULRt`jZAjtI-Rn*o*ZtJx6x2^`nQriWqS}I5EqLK
zq?^npxG|M?vFHsf5+}852V1krU~K1>K(bK}=H+~`6q4AdL9Jnyu_se3R?BV5){^@<
zu-dxxX_0E|>@H+-1EbEUnB`5aBnk)QFj2g^THHKcC~+3dXlb0s8B_Dd+~a+y{;|3M
z0;YN=HUE9l<G+;UU+2Fmf&3=^SAf6PU;Rt)w{z`#KIl)?SicJXwI=1yf=lmq|36o#
z{EG8yMZzCQ#P96WU+NQn75<g!{f96#?C-+=!TbId;a58E9|(r<|Ni;^8@cyaz+c(1
ze*hZ47hL?aKmP>$gDd+h%C8)yKTtlS{i9?5W-R?G`s*guAELT=zl;95W%Vn-UlW3V
zhywtu@1En=^x&`3e@(UgSsIJ{PtyNNy!<Nu*9G&R#p5Uef46-872sD#_Xh+H)9>B?
ze{p-ig8tQK{Q(I86uyK0-#yr`Sbv@Q{*3jL<<D5Z`N2Qt!EfSVzq!-*q8$Xl!@Ff(
JvHo`Ve*hOQGkpL6

literal 0
HcmV?d00001

diff --git a/tests/workbook/workbook_test_suite.cpp b/tests/workbook/workbook_test_suite.cpp
index 1c3ab1b5..be17357e 100644
--- a/tests/workbook/workbook_test_suite.cpp
+++ b/tests/workbook/workbook_test_suite.cpp
@@ -68,6 +68,7 @@ public:
         register_test(test_load_file);
         register_test(test_Issue279);
         register_test(test_Issue353);
+        register_test(test_Issue494);
     }
 
     void test_active_sheet()
@@ -509,5 +510,14 @@ public:
         xlnt_assert_equals(ws.row_properties(1).spans.get(), "1:8");
         xlnt_assert_equals(ws.row_properties(17).spans.get(), "2:7");
     }
+
+    void test_Issue494()
+    {
+        xlnt::workbook wb;
+        wb.load(path_helper::test_file("Issue494_shared_string.xlsx"));
+        auto ws = wb.active_sheet();
+        xlnt_assert_equals(ws.cell(2, 1).to_string(), "V1.00");
+        xlnt_assert_equals(ws.cell(2, 2).to_string(), "V1.00");
+    }
 };
 static workbook_test_suite x;