Exception thrown at this line when typeing to fast

[TobiasWallner]

Exception thrown at this line when typeing to fast

[flagarde]

@certik @TobiasWallner Should be fixed now

[TobiasWallner]

I just tried it and it works like a charm again, thx

[flagarde]

@certik @TobiasWallner I fully agree with all your comments. Sorry for this stupid change, will move back to string

[certik]

That code:

      m_Type = Type::CopyPaste;
      m_container.m_string.reset(new char[str.size() + 1]);
      std::copy(str.begin(), str.end(), m_container.m_string.get());
      m_container.m_string.get()[str.size()] = '\0';
      return;

should not be written like this. First of all, one should not use "new" by hand like this, since it usually leads to leaks and segfaults, possibly even causing the issue that you've seen. Second, as you said, this doesn't seem efficient. Why can't we simply append to the string?

[TobiasWallner]

Why can't we simply append to the string?

or assign

[certik]

@flagarde awesome, thanks! No worries, this should be easy to fix.

[flagarde]

I tested quickly on linux but please have a try too because string inside union it's a bit tricky