ObjFW  Diff

Differences From Artifact [d248ebf982]:

• File src/OFXMLAttribute.m — part of check-in [f3db074478] at 2011-09-17 19:18:09 on branch trunk — Make hashes independant of endianess in OF{Float,Double}{Vector,Matrix}.

  Also removes unnecessary OF_HASH_ADD_INT* macros (except
  OF_HASH_ADD_INT32 which was often used to hash hashes and is now called
  OF_HASH_ADD_HASH). (user: js, size: 3640) [annotate] [blame] [check-ins using]

To Artifact [04b01cd2af]:

• File src/OFXMLAttribute.m — part of check-in [e1e7ffa903] at 2011-09-22 23:25:42 on branch trunk — Exceptions are now autoreleased.

  This is safe as an "exception loop" can't happen, since if allocating
  an exception fails, it throws an OFAllocFailedException which is
  preallocated and can always be thrown.

  So, the worst case would be that an autorelease of an exception fails,
  triggering an OFOutOfMemoryException for which there is no memory,
  resulting in an OFAllocFailedException to be thrown. (user: js, size: 3649) [annotate] [blame] [check-ins using]

59
60
61
62
63
64
65
66

67
68
69
70
71
72
73
74

	self = [super init];

	@try {
		OFAutoreleasePool *pool = [[OFAutoreleasePool alloc] init];

		if (![[element name] isEqual: [self className]] ||
		    ![[element namespace] isEqual: OF_SERIALIZATION_NS])
			@throw [OFInvalidArgumentException newWithClass: isa

							       selector: _cmd];

		name = [[[element attributeForName: @"name"] stringValue]
		    copy];
		ns = [[[element attributeForName: @"namespace"] stringValue]
		    copy];
		stringValue = [[[element attributeForName: @"stringValue"]
		    stringValue] copy];

|
>
|

59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75

	self = [super init];

	@try {
		OFAutoreleasePool *pool = [[OFAutoreleasePool alloc] init];

		if (![[element name] isEqual: [self className]] ||
		    ![[element namespace] isEqual: OF_SERIALIZATION_NS])
			@throw [OFInvalidArgumentException
			    exceptionWithClass: isa
				      selector: _cmd];

		name = [[[element attributeForName: @"name"] stringValue]
		    copy];
		ns = [[[element attributeForName: @"namespace"] stringValue]
		    copy];
		stringValue = [[[element attributeForName: @"stringValue"]
		    stringValue] copy];