View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0005303JEDI VCL99 Otherpublic2010-08-18 11:112010-10-08 16:27
ReporterGreyWolf1983Assigned Toobones 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version3.38 
Target VersionFixed in Version 
Summary0005303: Error in example project JvComputerInfoExDemo.dpr
DescriptionAt line 226 of file JvComputerInfoExDemoForm.pas from folder JvComputerInfoEx from examples directory of JVCL installation, there are lines:
      case PropInfo^.PropType^.Kind of
        tkInteger, tkInt64:
          AddItem(Category, PropName, IntToStr(GetOrdProp(AnObject, PropName)), Indent);
, which should be replaced with
      case PropInfo^.PropType^.Kind of
        tkInteger:
          AddItem(Category, PropName, IntToStr(GetOrdProp(AnObject, PropName)), Indent);
        tkInt64:
          AddItem(Category, PropName, IntToStr(GetInt64Prop(AnObject, PropName)), Indent);
, because GetOrdProp (at least on Delphi 7), returns value of type int32, and int64 values do not fit into Result type.
This causes incorrect displaying of Memory info (TotalPhysicalMemory, TotalVirtualMemory, etc.) in the example project JvComputerInfoExDemo.exe (see attached screenshot).
TagsNo tags attached.

Activities

2010-08-18 11:11

 

bug.png (1,197 bytes)
bug.png (1,197 bytes)

obones

2010-08-20 11:41

administrator   ~0017590

Could you test this with the latest source code?

GreyWolf1983

2010-08-20 12:02

reporter   ~0017610

I browsed online SVN, and in version 3.39, in file
http://jvcl.svn.sourceforge.net/viewvc/jvcl/tags/JVCL3_39/examples/JvComputerInfoEx/JvComputerInfoExDemoForm.pas?revision=12600&view=markup
, code is the same at this point as in 3.38 release, and in
the latest revision from SVN:
http://jvcl.svn.sourceforge.net/viewvc/jvcl/trunk/jvcl/examples/JvComputerInfoEx/JvComputerInfoExDemoForm.pas?revision=12461&view=markup
code at this point is also the same.

outchy

2010-08-31 21:07

administrator   ~0017631

This issue was hidden in previous JVCL releases because of a bug in the JCL (it returned 2^32-1 instead of the real memory size when the size was over 4GB).

This is fixed in JVCL revision 12828.

Issue History

Date Modified Username Field Change
2010-08-18 11:11 GreyWolf1983 New Issue
2010-08-18 11:11 GreyWolf1983 File Added: bug.png
2010-08-20 11:41 obones Note Added: 0017590
2010-08-20 11:41 obones Status new => feedback
2010-08-20 12:02 GreyWolf1983 Note Added: 0017610
2010-08-31 21:07 outchy Note Added: 0017631
2010-10-08 16:27 obones Status feedback => resolved
2010-10-08 16:27 obones Resolution open => fixed
2010-10-08 16:27 obones Assigned To => obones