Re: commit - ap_Dialog_Insert_DateTime.cpp, m_iFormatIndex needed to be initialized in constructor, for Win32

Subject: Re: commit - ap_Dialog_Insert_DateTime.cpp, m_iFormatIndex needed to be initialized in constructor, for Win32
From: Paul Rohr (paul@abisource.com)
Date: Sat Jan 29 2000 - 12:42:24 CST

• sorted by: [ date ] [ thread ] [ subject ] [ author ]
• Next message: Robert Sievers: "Re: Commit -- line wrapping problem."
• Previous message: Paul Rohr: "reminder -- we need exporters *AND* importers"
• In reply to: Henrik Berg: "commit - ap_Dialog_Insert_DateTime.cpp, m_iFormatIndex needed to be initialized in constructor, for Win32"
• Next in thread: Henrik Berg: "RE: commit - ap_Dialog_Insert_DateTime.cpp, m_iFormatIndex needed to be initialized in constructor, for Win32"
• Next in thread: Leonard Rosenthol: "It's alive!!"
• Reply: Paul Rohr: "Re: commit - ap_Dialog_Insert_DateTime.cpp, m_iFormatIndex needed to be initialized in constructor, for Win32"
• Reply: Henrik Berg: "RE: commit - ap_Dialog_Insert_DateTime.cpp, m_iFormatIndex needed to be initialized in constructor, for Win32"

At 04:03 PM 1/29/00 +0100, Henrik Berg wrote:
>In the case of Win32, if you just pressed OK, the m_iFormatIndex was never
>chaged and assumed to point at the first entry, by it was not inited in
>constructor.

Henrik,

It looks like that field is declared in the base class, so would you mind
moving this fix to that constructor instead, to prevent similar problems on
other platforms?

Thanks,
Paul

• Next message: Robert Sievers: "Re: Commit -- line wrapping problem."
• Previous message: Paul Rohr: "reminder -- we need exporters *AND* importers"
• In reply to: Henrik Berg: "commit - ap_Dialog_Insert_DateTime.cpp, m_iFormatIndex needed to be initialized in constructor, for Win32"
• Next in thread: Henrik Berg: "RE: commit - ap_Dialog_Insert_DateTime.cpp, m_iFormatIndex needed to be initialized in constructor, for Win32"
• Next in thread: Leonard Rosenthol: "It's alive!!"
• Reply: Paul Rohr: "Re: commit - ap_Dialog_Insert_DateTime.cpp, m_iFormatIndex needed to be initialized in constructor, for Win32"
• Reply: Henrik Berg: "RE: commit - ap_Dialog_Insert_DateTime.cpp, m_iFormatIndex needed to be initialized in constructor, for Win32"

This archive was generated by hypermail 2b25 : Sat Jan 29 2000 - 12:37:03 CST