From: Garrett C. <yan...@gm...> - 2012-02-20 18:06:18
|
On Feb 20, 2012, at 2:51 AM, "Caspar Zhang" <su...@li...> wrote: > The branch, master, has been updated > via 6009edf814763bc17ee18769584c0b01f6108bca (commit) > from 2e2f72af82b25ea5604e670ee381d999fcfee5b2 (commit) > > > - Log ----------------------------------------------------------------- > commit 6009edf814763bc17ee18769584c0b01f6108bca > Author: Caspar Zhang <ca...@ca...> > Date: Mon Feb 20 18:42:36 2012 +0800 > > mm/lib: code cleanup > > Since there're a lot functions in mm/lib, some of them only used inside > the lib, some of them only used by specific tests, while the others are > general-purpose functions. I made a cleanup for the lib, so that a mm > case developer can write cases easier. No new functions added in this > patch. Hmmm... This doesn't really protect against someone abusing an internal API, or symbol pollution. Using the static keyword copiously and/or creating a _private.h header would make things a lot more explicit. Thanks, -Garrett |