On 2012/06/05 14:09:32, Peng wrote: > lgtm. But don't understand why origin code has problem. ...
11 years, 10 months ago
(2012-06-05 14:15:15 UTC)
#3
On 2012/06/05 14:09:32, Peng wrote:
> lgtm. But don't understand why origin code has problem. Please explain it.
> Thanks.
BTW, I already submit it. please close it.
Please see also http://cplusplus.com/reference/stl/vector/push_back/ > This effectively increases the vector size by one, which causes ...
11 years, 10 months ago
(2012-06-06 05:04:04 UTC)
#4
Please see also
http://cplusplus.com/reference/stl/vector/push_back/
> This effectively increases the vector size by one, which causes a reallocation
of the internal allocated
> storage if the vector size was equal to the vector capacity before the call.
Reallocations invalidate all
> previously obtained iterators, references and pointers.
By this error, Database::query returns NULL pointer, and ibus-pinyin crashes on
g_assert on Query::fill.
On 2012/06/06 05:04:04, hsumita1 wrote: > Please see also > > http://cplusplus.com/reference/stl/vector/push_back/ > > This ...
11 years, 10 months ago
(2012-06-06 14:18:58 UTC)
#5
On 2012/06/06 05:04:04, hsumita1 wrote:
> Please see also
>
> http://cplusplus.com/reference/stl/vector/push_back/
> > This effectively increases the vector size by one, which causes a
reallocation
> of the internal allocated
> > storage if the vector size was equal to the vector capacity before the call.
> Reallocations invalidate all
> > previously obtained iterators, references and pointers.
>
> By this error, Database::query returns NULL pointer, and ibus-pinyin crashes
on
> g_assert on Query::fill.
I see. Thank you.
Issue 6294043: Fixes a bug caused by a reallocation on vector.
(Closed)
Created 11 years, 10 months ago by hsumita1
Modified 11 years, 10 months ago
Reviewers: shawn.p.huang_gmail.com, Peng Huang, penghuang, Peng
Base URL: git://github.com/ibus/ibus-pinyin.git@master
Comments: 0