illumos gate

During browsing the code I have found these errors:

• nested calling of dgettext()
• unnecessary variable (buf[]) and snprintf()

This code (see here):

            char buf[256];
            int spa_version;

            VERIFY(zfs_spa_version(zhp, &spa_version) == 0);
            (void) snprintf(buf, sizeof (buf),
                dgettext(TEXT_DOMAIN, "Can't mount a version %lld " 
                "file system on a version %d pool. Pool must be" 
                " upgraded to mount this file system."),
                (u_longlong_t)zfs_prop_get_int(zhp,
                ZFS_PROP_VERSION), spa_version);
            zfs_error_aux(hdl, dgettext(TEXT_DOMAIN, buf));

should look like:

            int spa_version;

            VERIFY(zfs_spa_version(zhp, &spa_version) == 0);
            zfs_error_aux(hdl, dgettext(TEXT_DOMAIN,
                "Can't mount a version %lld file system on " 
                "a version %d pool. Pool must be upgraded to " 
                "mount this file system."),
                (u_longlong_t)zfs_prop_get_int(zhp,
                ZFS_PROP_VERSION), spa_version);