Am 28.06.2012 14:55, schrieb Peter Maydell: > Add functions deposit32(), deposit64(), extract32() and extract64() > to extract and deposit bitfields in 32 and 64 bit words. Based on > ideas by Jia Liu and Avi Kivity. > > Suggested-by: Jia Liu <pro...@gmail.com> > Suggested-by: Avi Kivity <a...@redhat.com> > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > Changes: > v1->v2: added missing brackets > v2->v3: renamed field32,field64 to extract32,extract64 > added deposit32,deposit64 at Avi's suggestion > fixed assertion as per Jay Foad's suggestion > bikeshed roof is now a slightly darker shade of grey
Reviewed-by: Andreas Färber <afaer...@suse.de> Small improvement would be to replace "Returns the" with "Returns: The" in line with how you annotated the arguments, and the function summary should go into its own paragraph between @foo: and Returns:. http://developer.gnome.org/gtk-doc-manual/unstable/documenting_symbols.html.en Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg