Я лично отказался от автообновления в ТП.

[quote author=Phoenix link=1185176115/0#0 date=1185176114]есть вероятность, что нагородил в коде.[/quote]
Да нет, нормально все.

[b]DmitrO[/b]
Дим, привет.
При отрисовке линеек может происходить нарушение границ массива слева (по отрицательным индексам).
Как это решить лучшим образом - не мог бы подсказать?

[quote]
void CCursorGridCtrl::PaintGridLines(CDC *pDC)
{
   int nStrokeCount;
   int nPointCount;
   int i;

   CRect rect = GetGridRect();

   int nFullVisibleRows = 0;
   if (m_bHorizontalLines)
      nFullVisibleRows = min(m_nVisibleRows, m_rows.GetSize() - m_nTopRowIndex); // может уходить в минус

   [b]nFullVisibleRows = max(nFullVisibleRows, 0);[/b] // есть ли более красивое решение?

   nStrokeCount = nFullVisibleRows;

// cut //
[/quote]
Может, стоит сбрасывать m_nTopRowIndex при очистке m_rows?

Phoenix
Да, это понятно и так запросто может быть, неверная отрисовка и падение из-за разрушения CRT кучи.
В принципе и причина и решение предложено выше, можно проверить.
Ну, возможно DmitrO предложит чуть более стройное по организации решение.
Т.к. Дима автор этого дела, а остальные - простые адепты тайных знаний


_{тайных равно недокументированных}

Или такой вариант:

void CCursorGridCtrl::ClearRows()
{
   OnChangeCurrentData(NULL);
   for (int i = 0; i < m_rows.GetSize(); i++)
       delete m_rows[i];
   m_rows.RemoveAll();

   m_bIsBOF = TRUE;
   m_bIsEOF = TRUE;

     m_nTopRowIndex = 0;
}


Phoenix
Ну, к примеру последний CVS build 2.5 с этим исправлением http://slil.ru/24655351

[quote author=kms link=1185176115/0#3 date=1185184087][quote author=Phoenix link=1185176115/0#0 date=1185176114]есть вероятность, что нагородил в коде.[/quote]
Да нет, нормально все.

[b]DmitrO[/b]
Дим, привет.
При отрисовке линеек может происходить нарушение границ массива слева (по отрицательным индексам).
Как это решить лучшим образом - не мог бы подсказать?

[quote]
void CCursorGridCtrl::PaintGridLines(CDC *pDC)
{
   int nStrokeCount;
   int nPointCount;
   int i;

   CRect rect = GetGridRect();

   int nFullVisibleRows = 0;
   if (m_bHorizontalLines)
      nFullVisibleRows = min(m_nVisibleRows, m_rows.GetSize() - m_nTopRowIndex); // может уходить в минус

   [b]nFullVisibleRows = max(nFullVisibleRows, 0);[/b] // есть ли более красивое решение?

   nStrokeCount = nFullVisibleRows;

// cut //
[/quote]
Может, стоит сбрасывать m_nTopRowIndex при очистке m_rows?[/quote]
Привет. Здесь никогда не должно уходить в минус. m_nTopRowIndex должен быть всегда >=0.

[quote]
Или такой вариант:

void CCursorGridCtrl::ClearRows()
{
   OnChangeCurrentData(NULL);
   for (int i = 0; i < m_rows.GetSize(); i++)
      delete m_rows[i];
   m_rows.RemoveAll();

   m_bIsBOF = TRUE;
   m_bIsEOF = TRUE;

   [b]m_nTopRowIndex = 0; [/b]
}
[/quote]
Вот тут действительно ошибка (то, что в оригинальном коде нет выделенной строки), в ClearRows дествительно надо занулять m_nTopRowIndex.

хотелось бы узнать какой-либо из описанных вариантов уже включен в ночной билд?
или пока использовать версию, выложенную в ветке?

kms писал(а) 31. Июля 2007 :: 20:33:

полностью согласен. сейчас подправленный тобой билд работает стабильно. хочу на след.неделе ВК обновить, вот и спрашиваю.