Paul Ishenin wrote:
Vincent wrote:
Renamed from UpdateFilterIndex to IntfFileTypeChanged.

VS> The TOpenFileDialogRec is a packed record. Why? It is better not to used 
packed
VS> records, unless you have a specific reason to dictate the memory layout.

Sorry, this is my internal love to packed records. For this case I
think there is no difference between them. Changed to record.

As a general note, even if the size is the same in this case, the packed also dictates on how a variable of this type is placed on stack. While a normal record is placed on a dword (or was it the first member) boundary, a packed record is immediately placed after the previous variable. This can lead to serious speed penalties and/or buserrors on platforms not supporting unaligned data access (sparc/arm)

Because of this we have removed all unnecesary packed declarations

Marc

_________________________________________________________________
    To unsubscribe: mail [EMAIL PROTECTED] with
               "unsubscribe" as the Subject
  archives at http://www.lazarus.freepascal.org/mailarchives

Reply via email to