Reviewed-by: Jeremy Huddleston <jerem...@apple.com> On Nov 24, 2011, at 3:47 PM, Tobias Droste wrote:
> Status is defined as int -> NULL is not a valid return value > > v2: Remove unneeded cast to Status > > Signed-off-by: Tobias Droste <tdro...@gmx.de> > --- > src/XpNotifyPdm.c | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/src/XpNotifyPdm.c b/src/XpNotifyPdm.c > index c1ceb8e..c77f9e1 100644 > --- a/src/XpNotifyPdm.c > +++ b/src/XpNotifyPdm.c > @@ -231,7 +231,7 @@ XpGetPdmStartParams ( > /* > * Error - cannot determine or establish a selection_display. > */ > - return( (Status) NULL ); > + return( 0 ); > } > > /* > @@ -268,7 +268,7 @@ XpGetPdmStartParams ( > XCloseDisplay( *selection_display ); > *selection_display = (Display *) NULL; > } > - return( (Status) NULL ); > + return( 0 ); > } > > status = XmbTextListToTextProperty( *selection_display, list, 6, > @@ -283,7 +283,7 @@ XpGetPdmStartParams ( > XCloseDisplay( *selection_display ); > *selection_display = (Display *) NULL; > } > - return( (Status) NULL ); > + return( 0 ); > } > > *type = text_prop.encoding; > @@ -293,7 +293,7 @@ XpGetPdmStartParams ( > > XFree(list[5]); > > - return( (Status) 1 ); > + return( 1 ); > } > > /****************************************************************************** > -- > 1.7.3.4 > > _______________________________________________ > xorg-devel@lists.x.org: X.Org development > Archives: http://lists.x.org/archives/xorg-devel > Info: http://lists.x.org/mailman/listinfo/xorg-devel > _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel