Hello community,

here is the log from the commit of package libvdpau for openSUSE:Factory 
checked in at 2013-10-29 09:39:21
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/libvdpau (Old)
 and      /work/SRC/openSUSE:Factory/.libvdpau.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "libvdpau"

Changes:
--------
--- /work/SRC/openSUSE:Factory/libvdpau/libvdpau.changes        2013-03-25 
20:40:37.000000000 +0100
+++ /work/SRC/openSUSE:Factory/.libvdpau.new/libvdpau.changes   2013-10-29 
09:39:22.000000000 +0100
@@ -1,0 +2,7 @@
+Sun Oct 27 09:25:25 UTC 2013 - [email protected]
+
+- update to vdpauinfo 0.1
+  * This release fixes a problem where ranges were queried for
+    mixer parameters and attributes where ranges were not allowed.
+
+-------------------------------------------------------------------

Old:
----
  vdpauinfo-0.0.6.tar.gz

New:
----
  vdpauinfo-0.1.tar.gz

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ libvdpau.spec ++++++
--- /var/tmp/diff_new_pack.bS3Y9p/_old  2013-10-29 09:39:23.000000000 +0100
+++ /var/tmp/diff_new_pack.bS3Y9p/_new  2013-10-29 09:39:23.000000000 +0100
@@ -24,7 +24,7 @@
 Group:          System/Libraries
 Url:            http://people.freedesktop.org/~aplattner
 Source:         
http://people.freedesktop.org/~aplattner/vdpau/%{name}-%{version}.tar.gz
-Source1:        vdpauinfo-0.0.6.tar.gz
+Source1:        vdpauinfo-0.1.tar.gz
 Source2:        README
 Source99:       baselibs.conf
 Source100:      %{name}-rpmlintrc

++++++ vdpauinfo-0.0.6.tar.gz -> vdpauinfo-0.1.tar.gz ++++++
++++ 6491 lines of diff (skipped)
++++    retrying with extended exclude list
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' --exclude Makefile.in --exclude configure --exclude 
config.guess --exclude '*.pot' --exclude mkinstalldirs --exclude aclocal.m4 
--exclude config.sub --exclude depcomp --exclude install-sh --exclude ltmain.sh 
old/vdpauinfo-0.0.6/AUTHORS new/vdpauinfo-0.1/AUTHORS
--- old/vdpauinfo-0.0.6/AUTHORS 2009-09-01 23:08:12.000000000 +0200
+++ new/vdpauinfo-0.1/AUTHORS   2012-11-20 18:54:49.000000000 +0100
@@ -1,2 +1,5 @@
 Wladimir J. van der Laan
 Aaron Plattner
+Rajib Mahapatra
+José Hiram Soltren
+Andy Ritger
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' --exclude Makefile.in --exclude configure --exclude 
config.guess --exclude '*.pot' --exclude mkinstalldirs --exclude aclocal.m4 
--exclude config.sub --exclude depcomp --exclude install-sh --exclude ltmain.sh 
old/vdpauinfo-0.0.6/ChangeLog new/vdpauinfo-0.1/ChangeLog
--- old/vdpauinfo-0.0.6/ChangeLog       2009-09-17 20:38:59.000000000 +0200
+++ new/vdpauinfo-0.1/ChangeLog 2013-10-22 23:51:38.000000000 +0200
@@ -1,3 +1,120 @@
+commit a7cc3fa5760b8915853223f487f054c9100c2795
+Author: Aaron Plattner <[email protected]>
+Date:   Tue Oct 22 14:48:38 2013 -0700
+
+    Bump version to 0.1
+    
+    I'd like to finish dealing with the TODO items in vdpauinfo.cpp before 
declaring
+    this 1.0.
+    
+    Signed-off-by: Aaron Plattner <[email protected]>
+
+commit e14ecabc6225a3333360bbe88903d2dce8e17056
+Author: Aaron Plattner <[email protected]>
+Date:   Mon Oct 21 14:48:38 2013 -0700
+
+    vdpauinfo: remove is_supported initializers
+    
+    These variables were initialized to work around a bug in the NVIDIA driver 
where
+    the is_supported output parameter was only initialized if the feature being
+    queried was not supported.  This was fixed in the 190.* driver release 
series,
+    making the workaround unnecessary.
+    
+    Signed-off-by: Aaron Plattner <[email protected]>
+    Reviewed-by: José Hiram Soltren <[email protected]>
+
+commit 6c582869ecb04e02a2f47f245bb524b6d9b7bf07
+Author: Aaron Plattner <[email protected]>
+Date:   Mon Oct 21 14:12:23 2013 -0700
+
+    vdpauinfo: remove TODO regarding INVALID_CHROMA_TYPE when querying 444
+    
+    This was a bug in the NVIDIA driver that was fixed in the 256.44 release 
with
+    the following changelog entry:
+    
+        * Fixed VDPAU to not print a debug error message when calling
+          VdpVideoMixerQueryFeatureSupport with an unsupported or unknown
+          VdpVideoMixerFeature.
+    
+    Signed-off-by: Aaron Plattner <[email protected]>
+    Reviewed-by: José Hiram Soltren <[email protected]>
+
+commit 0999893597059a31b99b3085882969aca3f31207
+Author: Aaron Plattner <[email protected]>
+Date:   Fri Dec 14 08:36:51 2012 -0800
+
+    Remove redundant LICENSE file
+    
+    The same license text is in the more canonical COPYING file.
+    
+    Signed-off-by: Aaron Plattner <[email protected]>
+
+commit d9fbee39b8240dd679de139be8dfc2120228d04f
+Author: Aaron Plattner <[email protected]>
+Date:   Tue Nov 20 09:55:34 2012 -0800
+
+    Add Andy to the AUTHORS list
+    
+    Signed-off-by: Aaron Plattner <[email protected]>
+
+commit bb73db54c3611d20babdb098aefa6d69a4e8c684
+Author: Andy Ritger <[email protected]>
+Date:   Fri Nov 16 15:40:40 2012 -0800
+
+    vdpauinfo: use rgb_types[x].id, not chroma_types[x].id, as the 
VdpRGBAFormat argument to VdpOutputSurfaceQueryPutBitsYCbCrCapabilities().
+    
+    Signed-off-by: Andy Ritger <[email protected]>
+    Reviewed-by: Stephen Warren <[email protected]>
+    Reviewed-by: Jose Soltren <[email protected]>
+    Signed-off-by: Aaron Plattner <[email protected]>
+
+commit df16fd8e83b36e6e5bdb0e73ff57cb432d113a28
+Author: Aaron Plattner <[email protected]>
+Date:   Wed Nov 14 17:13:11 2012 -0800
+
+    Remove the version number from vdpauinfo.cpp
+    
+    The version is defined in the AC_INIT call in configure.ac, and doesn't 
need to
+    be duplicated into other source files where it will quickly grow out of 
date
+    (e.g. here, where it said 0.0.2 when the version is really 0.0.6).
+
+commit 9041dc8fc10312967e9398912a0d655d9cccff72
+Merge: fb75470 f2ced6a
+Author: Aaron Plattner <[email protected]>
+Date:   Wed Nov 14 16:59:34 2012 -0800
+
+    Merge fix for improper attribute queries
+    
+    Tested-by: Aaron Plattner <[email protected]>
+    Signed-off-by: Aaron Plattner <[email protected]>
+
+commit f2ced6afa0d01698745cc91710def89b670e3d56
+Author: José Hiram Soltren <[email protected]>
+Date:   Wed Nov 14 15:36:25 2012 -0800
+
+    Whitespace cleanup to vdpauinfo.cpp
+
+commit 30f825629c72613ee9b86c8e05173e884d8ee2e9
+Author: José Hiram Soltren <[email protected]>
+Date:   Wed Nov 14 15:09:16 2012 -0800
+
+    Add José Soltren to AUTHORS.
+
+commit 09a467217ed3347a30c3899dace68bc24021af3f
+Author: José Hiram Soltren <[email protected]>
+Date:   Wed Nov 14 15:06:49 2012 -0800
+
+    Only query ranges for appropriate VDPAU attributes
+    
+    Update vdpauinfo.cpp to not query values from VDPAU which
+    do not support ranges. See vdpau.h for more information.
+
+commit fb754701e83d565e4333ce7d3fce216194b40266
+Author: Aaron Plattner <[email protected]>
+Date:   Thu Sep 24 12:39:50 2009 -0700
+
+    Add Rajib to the AUTHORS list
+
 commit a84c610c0b30a1a1e86f8b838dc90d20e23ddb83
 Author: Rajib Mahapatra <[email protected]>
 Date:   Mon Sep 14 14:01:38 2009 -0700
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' --exclude Makefile.in --exclude configure --exclude 
config.guess --exclude '*.pot' --exclude mkinstalldirs --exclude aclocal.m4 
--exclude config.sub --exclude depcomp --exclude install-sh --exclude ltmain.sh 
old/vdpauinfo-0.0.6/configure.ac new/vdpauinfo-0.1/configure.ac
--- old/vdpauinfo-0.0.6/configure.ac    2009-09-14 23:00:11.000000000 +0200
+++ new/vdpauinfo-0.1/configure.ac      2013-10-22 23:51:27.000000000 +0200
@@ -1,5 +1,5 @@
 AC_PREREQ([2.57])
-AC_INIT(vdpauinfo,[0.0.6],[[email protected]],vdpauinfo)
+AC_INIT(vdpauinfo,[0.1],[[email protected]],vdpauinfo)
 AM_INIT_AUTOMAKE([dist-bzip2])
 AM_MAINTAINER_MODE
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' --exclude Makefile.in --exclude configure --exclude 
config.guess --exclude '*.pot' --exclude mkinstalldirs --exclude aclocal.m4 
--exclude config.sub --exclude depcomp --exclude install-sh --exclude ltmain.sh 
old/vdpauinfo-0.0.6/missing new/vdpauinfo-0.1/missing
--- old/vdpauinfo-0.0.6/missing 2009-09-17 20:36:52.000000000 +0200
+++ new/vdpauinfo-0.1/missing   2012-12-14 17:36:27.000000000 +0100
@@ -1,10 +1,10 @@
 #! /bin/sh
 # Common stub for a few missing GNU programs while installing.
 
-scriptversion=2006-05-10.23
+scriptversion=2012-01-06.13; # UTC
 
-# Copyright (C) 1996, 1997, 1999, 2000, 2002, 2003, 2004, 2005, 2006
-#   Free Software Foundation, Inc.
+# Copyright (C) 1996, 1997, 1999, 2000, 2002, 2003, 2004, 2005, 2006,
+# 2008, 2009, 2010, 2011, 2012 Free Software Foundation, Inc.
 # Originally by Fran,cois Pinard <[email protected]>, 1996.
 
 # This program is free software; you can redistribute it and/or modify
@@ -18,9 +18,7 @@
 # GNU General Public License for more details.
 
 # You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
-# 02110-1301, USA.
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 # As a special exception to the GNU General Public License, if you
 # distribute this file as part of a program that contains a
@@ -86,9 +84,11 @@
   help2man     touch the output file
   lex          create \`lex.yy.c', if possible, from existing .c
   makeinfo     touch the output file
-  tar          try tar, gnutar, gtar, then tar without non-portable flags
   yacc         create \`y.tab.[ch]', if possible, from existing .[ch]
 
+Version suffixes to PROGRAM as well as the prefixes \`gnu-', \`gnu', and
+\`g' are ignored when checking the name.
+
 Send bug reports to <[email protected]>."
     exit $?
     ;;
@@ -106,23 +106,21 @@
 
 esac
 
+# normalize program name to check for.
+program=`echo "$1" | sed '
+  s/^gnu-//; t
+  s/^gnu//; t
+  s/^g//; t'`
+
 # Now exit if we have it, but it failed.  Also exit now if we
 # don't have it and --version was passed (most likely to detect
-# the program).
+# the program).  This is about non-GNU programs, so use $1 not
+# $program.
 case $1 in
-  lex|yacc)
+  lex*|yacc*)
     # Not GNU programs, they don't have --version.
     ;;
 
-  tar)
-    if test -n "$run"; then
-       echo 1>&2 "ERROR: \`tar' requires --run"
-       exit 1
-    elif test "x$2" = "x--version" || test "x$2" = "x--help"; then
-       exit 1
-    fi
-    ;;
-
   *)
     if test -z "$run" && ($1 --version) > /dev/null 2>&1; then
        # We have it, but it failed.
@@ -138,7 +136,7 @@
 
 # If it does not exist, or fails to run (possibly an outdated version),
 # try to emulate it.
-case $1 in
+case $program in
   aclocal*)
     echo 1>&2 "\
 WARNING: \`$1' is $msg.  You should only need it if
@@ -148,7 +146,7 @@
     touch aclocal.m4
     ;;
 
-  autoconf)
+  autoconf*)
     echo 1>&2 "\
 WARNING: \`$1' is $msg.  You should only need it if
          you modified \`${configure_ac}'.  You might want to install the
@@ -157,7 +155,7 @@
     touch configure
     ;;
 
-  autoheader)
+  autoheader*)
     echo 1>&2 "\
 WARNING: \`$1' is $msg.  You should only need it if
          you modified \`acconfig.h' or \`${configure_ac}'.  You might want
@@ -187,7 +185,7 @@
           while read f; do touch "$f"; done
     ;;
 
-  autom4te)
+  autom4te*)
     echo 1>&2 "\
 WARNING: \`$1' is needed, but is $msg.
          You might have modified some files without having the
@@ -210,7 +208,7 @@
     fi
     ;;
 
-  bison|yacc)
+  bison*|yacc*)
     echo 1>&2 "\
 WARNING: \`$1' $msg.  You should only need it if
          you modified a \`.y' file.  You may need the \`Bison' package
@@ -218,7 +216,7 @@
          \`Bison' from any GNU archive site."
     rm -f y.tab.c y.tab.h
     if test $# -ne 1; then
-        eval LASTARG="\${$#}"
+        eval LASTARG=\${$#}
        case $LASTARG in
        *.y)
            SRCFILE=`echo "$LASTARG" | sed 's/y$/c/'`
@@ -240,7 +238,7 @@
     fi
     ;;
 
-  lex|flex)
+  lex*|flex*)
     echo 1>&2 "\
 WARNING: \`$1' is $msg.  You should only need it if
          you modified a \`.l' file.  You may need the \`Flex' package
@@ -248,7 +246,7 @@
          \`Flex' from any GNU archive site."
     rm -f lex.yy.c
     if test $# -ne 1; then
-        eval LASTARG="\${$#}"
+        eval LASTARG=\${$#}
        case $LASTARG in
        *.l)
            SRCFILE=`echo "$LASTARG" | sed 's/l$/c/'`
@@ -263,7 +261,7 @@
     fi
     ;;
 
-  help2man)
+  help2man*)
     echo 1>&2 "\
 WARNING: \`$1' is $msg.  You should only need it if
         you modified a dependency of a manual page.  You may need the
@@ -277,11 +275,11 @@
     else
        test -z "$file" || exec >$file
        echo ".ab help2man is required to generate this page"
-       exit 1
+       exit $?
     fi
     ;;
 
-  makeinfo)
+  makeinfo*)
     echo 1>&2 "\
 WARNING: \`$1' is $msg.  You should only need it if
          you modified a \`.texi' or \`.texinfo' file, or any other file
@@ -310,41 +308,6 @@
     touch $file
     ;;
 
-  tar)
-    shift
-
-    # We have already tried tar in the generic part.
-    # Look for gnutar/gtar before invocation to avoid ugly error
-    # messages.
-    if (gnutar --version > /dev/null 2>&1); then
-       gnutar "$@" && exit 0
-    fi
-    if (gtar --version > /dev/null 2>&1); then
-       gtar "$@" && exit 0
-    fi
-    firstarg="$1"
-    if shift; then
-       case $firstarg in
-       *o*)
-           firstarg=`echo "$firstarg" | sed s/o//`
-           tar "$firstarg" "$@" && exit 0
-           ;;
-       esac
-       case $firstarg in
-       *h*)
-           firstarg=`echo "$firstarg" | sed s/h//`
-           tar "$firstarg" "$@" && exit 0
-           ;;
-       esac
-    fi
-
-    echo 1>&2 "\
-WARNING: I can't seem to be able to run \`tar' with the given arguments.
-         You may want to install GNU tar or Free paxutils, or check the
-         command line arguments."
-    exit 1
-    ;;
-
   *)
     echo 1>&2 "\
 WARNING: \`$1' is needed, and is $msg.
@@ -363,5 +326,6 @@
 # eval: (add-hook 'write-file-hooks 'time-stamp)
 # time-stamp-start: "scriptversion="
 # time-stamp-format: "%:y-%02m-%02d.%02H"
-# time-stamp-end: "$"
+# time-stamp-time-zone: "UTC"
+# time-stamp-end: "; # UTC"
 # End:
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' --exclude Makefile.in --exclude configure --exclude 
config.guess --exclude '*.pot' --exclude mkinstalldirs --exclude aclocal.m4 
--exclude config.sub --exclude depcomp --exclude install-sh --exclude ltmain.sh 
old/vdpauinfo-0.0.6/vdpauinfo.cpp new/vdpauinfo-0.1/vdpauinfo.cpp
--- old/vdpauinfo-0.0.6/vdpauinfo.cpp   2009-09-14 22:58:32.000000000 +0200
+++ new/vdpauinfo-0.1/vdpauinfo.cpp     2013-10-22 23:51:27.000000000 +0200
@@ -1,6 +1,5 @@
-/* 
-Query and display NVIDIA VDPAU capabilities, a la glxinfo 
-version 0.0.2
+/*
+Query and display NVIDIA VDPAU capabilities, a la glxinfo
 
 Copyright (c) 2008 Wladimir J. van der Laan
 
@@ -26,8 +25,6 @@
 OTHER DEALINGS IN THE SOFTWARE.
 */
 /// TODO
-/// - possible bug, querying VDP_CHROMA_TYPE_444 returns an error
-///  (VDP_STATUS_INVALID_CHROMA_TYPE) is this NVIDIA or us?
 /// - parse display/screen from command line
 /// - list color table formats for queryOutputSurface
 
@@ -48,7 +45,7 @@
        device->GetInformationString(&info) != VDP_STATUS_OK)
     {
         fprintf(stderr, "Error querying API version or information\n");
-        exit(-1);    
+        exit(-1);
     }
     printf("API version: %i\n", api);
     printf("Information string: %s\n", info);
@@ -111,19 +108,18 @@
     printf("-------------------------------------------\n");
     for(int x=0; x<chroma_type_count; ++x)
     {
-        VdpBool is_supported = false;
+        VdpBool is_supported;
         uint32_t max_width, max_height;
 
-        rv = device->VideoSurfaceQueryCapabilities(device->device, 
chroma_types[x].id, 
+        rv = device->VideoSurfaceQueryCapabilities(device->device, 
chroma_types[x].id,
             &is_supported, &max_width, &max_height);
         if(rv == VDP_STATUS_OK && is_supported)
         {
-            printf("%-6s %5i %5i  ", chroma_types[x].name, 
+            printf("%-6s %5i %5i  ", chroma_types[x].name,
                 max_width, max_height);
             /* Find out supported formats */
             for(int y=0; y<ycbcr_type_count; ++y)
             {
-                is_supported = false;
                 rv = device->VideoSurfaceQueryGetPutBitsYCbCrCapabilities(
                     device->device, chroma_types[x].id, ycbcr_types[y].id,
                     &is_supported);
@@ -146,23 +142,22 @@
     printf("----------------------------------------------------\n");
     for(int x=0; x<rgb_type_count; ++x)
     {
-        VdpBool is_supported, native=false;
+        VdpBool is_supported, native;
         uint32_t max_width, max_height;
 
-        rv = device->OutputSurfaceQueryCapabilities(device->device, 
rgb_types[x].id, 
+        rv = device->OutputSurfaceQueryCapabilities(device->device, 
rgb_types[x].id,
             &is_supported, &max_width, &max_height);
         device->OutputSurfaceQueryGetPutBitsNativeCapabilities(device->device, 
rgb_types[x].id,
             &native);
         if(rv == VDP_STATUS_OK && is_supported)
         {
-            printf("%-16s %5i %5i    %c  ", rgb_types[x].name, 
+            printf("%-16s %5i %5i    %c  ", rgb_types[x].name,
                 max_width, max_height, native?'y':'-');
             /* Find out supported formats */
             for(int y=0; y<ycbcr_type_count; ++y)
             {
-                is_supported = false;
                 rv = device->OutputSurfaceQueryPutBitsYCbCrCapabilities(
-                    device->device, chroma_types[x].id, ycbcr_types[y].id,
+                    device->device, rgb_types[x].id, ycbcr_types[y].id,
                     &is_supported);
                 if(rv == VDP_STATUS_OK && is_supported)
                 {
@@ -188,11 +183,11 @@
         VdpBool is_supported;
         uint32_t max_width, max_height;
 
-        rv = device->BitmapSurfaceQueryCapabilities(device->device, 
rgb_types[x].id, 
+        rv = device->BitmapSurfaceQueryCapabilities(device->device, 
rgb_types[x].id,
             &is_supported, &max_width, &max_height);
         if(rv == VDP_STATUS_OK && is_supported)
         {
-            printf("%-16s %5i %5i\n", rgb_types[x].name, 
+            printf("%-16s %5i %5i\n", rgb_types[x].name,
                 max_width, max_height);
         }
     }
@@ -266,13 +261,14 @@
     printf("------------------------------------\n");
     for(int x=0; x<mixer_features_count; ++x)
     {
-        VdpBool is_supported = true; /* There seems to be a bug in 
VideoMixerQueryFeatureSupport, is_supported is only set if the feature is not 
supported */
-        rv = device->VideoMixerQueryFeatureSupport(device->device, 
mixer_features[x].id, 
+        VdpBool is_supported;
+
+        rv = device->VideoMixerQueryFeatureSupport(device->device, 
mixer_features[x].id,
             &is_supported);
         is_supported = (rv == VDP_STATUS_OK && is_supported);
-        printf("%-32s %c\n", mixer_features[x].name, 
+        printf("%-32s %c\n", mixer_features[x].name,
             is_supported?'y':'-');
-       
+
     }
     printf("\n");
     // Parameters (+range)
@@ -280,17 +276,18 @@
     printf("-----------------------------------------------------\n");
     for(int x=0; x<mixer_parameters_count; ++x)
     {
-        VdpBool is_supported = false;
+        VdpBool is_supported;
 
-        rv = device->VideoMixerQueryParameterSupport(device->device, 
mixer_parameters[x].id, 
+        rv = device->VideoMixerQueryParameterSupport(device->device, 
mixer_parameters[x].id,
             &is_supported);
         is_supported = (rv == VDP_STATUS_OK && is_supported);
-        printf("%-32s %c  ", mixer_parameters[x].name, 
+        printf("%-32s %c  ", mixer_parameters[x].name,
             is_supported?'y':'-');
-        if(is_supported)
+        /* VDPAU spec does not allow range query for DT_NONE types */
+        if(is_supported && mixer_parameters[x].aux != DT_NONE)
         {
             uint32_t minval, maxval;
-            rv = device->VideoMixerQueryParameterValueRange(device->device, 
mixer_parameters[x].id, 
+            rv = device->VideoMixerQueryParameterValueRange(device->device, 
mixer_parameters[x].id,
                 (void*)&minval, (void*)&maxval);
             if(rv == VDP_STATUS_OK)
                 display_range(mixer_parameters[x].aux, minval, maxval);
@@ -298,30 +295,31 @@
         printf("\n");
     }
     printf("\n");
-    
+
     // Attributes (+range)
     printf("attribute name                  sup      min      max\n");
     printf("-----------------------------------------------------\n");
     for(int x=0; x<mixer_attributes_count; ++x)
     {
-        VdpBool is_supported = false;
+        VdpBool is_supported;
 
-        rv = device->VideoMixerQueryAttributeSupport(device->device, 
mixer_attributes[x].id, 
+        rv = device->VideoMixerQueryAttributeSupport(device->device, 
mixer_attributes[x].id,
             &is_supported);
         is_supported = (rv == VDP_STATUS_OK && is_supported);
-        printf("%-32s %c  ", mixer_attributes[x].name, 
+        printf("%-32s %c  ", mixer_attributes[x].name,
             is_supported?'y':'-');
-        if(is_supported)
+        /* VDPAU spec does not allow range query for DT_NONE types */
+        if(is_supported && mixer_attributes[x].aux != DT_NONE)
         {
             uint32_t minval, maxval;
-            rv = device->VideoMixerQueryAttributeValueRange(device->device, 
mixer_parameters[x].id, 
+            rv = device->VideoMixerQueryAttributeValueRange(device->device, 
mixer_parameters[x].id,
                 (void*)&minval, (void*)&maxval);
             if(rv == VDP_STATUS_OK)
                 display_range(mixer_attributes[x].aux, minval, maxval);
         }
         printf("\n");
     }
-    printf("\n");    
+    printf("\n");
 }
 
 /******************* Decoder ****************/
@@ -357,14 +355,14 @@
     printf("-------------------------------------------\n");
     for(int x=0; x<decoder_profile_count; ++x)
     {
-        VdpBool is_supported = false;
+        VdpBool is_supported;
         uint32_t max_level, max_macroblocks, max_width, max_height;
 
-        rv = device->DecoderQueryCapabilities(device->device, 
decoder_profiles[x].id, 
+        rv = device->DecoderQueryCapabilities(device->device, 
decoder_profiles[x].id,
             &is_supported, &max_level, &max_macroblocks, &max_width, 
&max_height);
         if(rv == VDP_STATUS_OK && is_supported)
         {
-            printf("%-20s %2i %5i %5i %5i\n", decoder_profiles[x].name, 
+            printf("%-20s %2i %5i %5i %5i\n", decoder_profiles[x].name,
                 max_level, max_macroblocks, max_width, max_height);
         }
     }
@@ -385,7 +383,7 @@
     }
     screen = DefaultScreen(display);
     printf("display: %s   screen: %i\n", display_name, screen);
-    
+
     /* Create device */
     VdpDevice device;
     VdpGetProcAddress *get_proc_address;
@@ -395,10 +393,10 @@
     {
         fprintf(stderr, "Error creating VDPAU device: %i\n", rv); /* cannot 
use GetErrorString here */
         exit(-1);
-    }    
-    
+    }
+
     VDPDeviceImpl *impl = new VDPDeviceImpl(device, get_proc_address);
-    
+
     queryBaseInfo(impl);
     queryVideoSurface(impl);
     queryDecoderCaps(impl);
@@ -408,4 +406,3 @@
 
     printf("\n");
 }
-

-- 
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to