jirentabu / crashrpt

Automatically exported from code.google.com/p/crashrpt
0 stars 0 forks source link

Order of custom properties #131

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Create a crash report with some custom properties
2. Create another crash report, this time just add some more properties

What is the expected output? What do you see instead?
The order of the custom properties must be the order in which they are added.
However, their order seems to always be reversed.
For example:
If I have the following code:

  crAddProperty("A", "1")
  crAddProperty("B", "2")
  crAddProperty("C", "3")

then the order of the properties in the XML (and how to get it with crprober 
/get XmlDescCustomProps PropertyValue) is:

  Property C = 0
  Property B = 1
  Property A = 2

This becomes a problem if you then add another property later on, say 'D', then 
all the ones A through C will need to be incremented.

What version of CrashRpt are you using?
1.3.0

What is your version of Visual Studio?
2010

Original issue reported on code.google.com by cron...@gmail.com on 6 Feb 2012 at 12:43

GoogleCodeExporter commented 9 years ago
OK I only now realised you can build a map by getting the 'index' of the 
property name.
So maybe you can close this issue, but augment the documentation?

Original comment by cron...@gmail.com on 6 Feb 2012 at 1:21

GoogleCodeExporter commented 9 years ago

Original comment by zexspect...@gmail.com on 7 Feb 2012 at 5:16

GoogleCodeExporter commented 9 years ago
I added the following to the doxygen documentation block of crAddPropertyW() 
(trunk, CrashRpt.h: line 911):

"In the XML file properties are ordered by names in alphabetic order."

Let me know if it is still unclear and something else should be added.

Original comment by zexspect...@gmail.com on 26 Aug 2012 at 6:31

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r1357.

Original comment by zexspect...@gmail.com on 26 Aug 2012 at 6:32