This patch is WITHDRAWN.
As pointed out by Scott Duplichan, the additional entries in arith.h are not 
used by existing code and are of no use outside of the "private" gdtoa library.

These changes will NOT be committed to SVN.

My most sincere thanks to everyone that has provided review comments.

Daryl McDaniel

"It is the mark of an educated mind to be able to entertain a thought without 
accepting it."
- Aristotle


-----Original Message-----
From: Scott Duplichan [mailto:[email protected]] 
Sent: Monday, August 18, 2014 9:21 PM
To: [email protected]
Subject: Re: [edk2] Patch for review: StdLibPrivateInternalFiles/Include/{Ia32, 
X64}/arith.h

Mcdaniel, Daryl [mailto:[email protected]] wrote:

]Please review the attached changes.
]
]StdLibPrivateInternalFiles/Include/{Ia32,X64}/arith.h: Use programmatically 
]generated arith.h files for greater compatibility with the gdtoa library in 
]StdLib.
]
]Contributed-under: TianoCore Contribution Agreement 1.0
]Signed-off-by: Daryl McDaniel <[email protected]>
]Reviewed-by: 
]
]Thank You,
]Daryl McDaniel

Hello Daryl,

In what way does this change provide greater compatibility with the gdtoa 
library? The gdtoa library is documented as for internal use by stdlib. Yet 
stdlib doesn't use the new defines added by this patch.

Thanks,
Scott


------------------------------------------------------------------------------
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-devel

------------------------------------------------------------------------------
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to