1. 27 3月, 2008 8 次提交
    • J
      lxcError: mark a string and add to the list of nearly-checked functions · c5adab52
      Jim Meyering 提交于
      * Makefile.maint (msg_gen_function): Add, but commented-out.
      * src/lxc_conf.c (lxcParseXML): Mark a diagnostic for translation.
      c5adab52
    • J
      testError: mark most string arguments for translation · 76a70984
      Jim Meyering 提交于
      * Makefile.maint (msg_gen_function): Add testError
      * src/test.c (testDomainSave, testDomainRestore): Mark strings.
      (testDomainCoreDump): Likewise.
      76a70984
    • J
      openvzLog: mark its string arguments for translation · 7f863926
      Jim Meyering 提交于
      * Makefile.maint (msg_gen_function): Add openvzLog.
      * src/openvz_conf.c (openvzAssignVMDef, openvzParseXML): Mark strings.
      * src/openvz_driver.c (openvzDomainShutdown, openvzDomainReboot):
      (openvzDomainDefineXML, openvzDomainCreateLinux):
      (openvzDomainCreate, openvzDomainUndefine): Likewise.
      7f863926
    • J
      error: mark most string arguments for translation · 5bb81e1d
      Jim Meyering 提交于
      Also mark some arguments to __virRaiseError.
      * Makefile.maint (msg_gen_function): Add error.
      * po/POTFILES.in: Add src/openvz_conf.c and src/openvz_driver.c.
      * src/openvz_conf.c (openvzParseXML, openvzGetVPSInfo): Mark strings.
      * src/openvz_driver.c (openvzDomainLookupByID): Likewise.
      (openvzDomainLookupByUUID, openvzDomainLookupByName): Likewise.
      (openvzDomainGetInfo, openvzDomainShutdown, openvzDomainReboot)
      (openvzDomainCreate, openvzDomainUndefine): Likewise.
      * src/remote_internal.c (check_cert_file): Likewise.
      (negotiate_gnutls_on_connection, verify_certificate): Likewise.
      (remoteAuthenticate, addrToString, remoteAuthSASL): Likewise.
      (remoteAuthPolkit, call, really_read_buf): Likewise.
      5bb81e1d
    • J
      qemudReportError: mark for translation string args to this function · b53c4e51
      Jim Meyering 提交于
      * Makefile.maint (msg_gen_function): Add qemudReportError.
      * src/qemu_conf.c (qemudLoadDriverConfig)
      (qemudExtractVersion, qemudParseDiskXML, qemudParseInterfaceXML)
      (qemudParseInputXML, qemudParseXML, qemudNetworkIfaceConnect)
      (qemudBuildCommandLine, qemudSaveConfig, qemudParseVMDeviceDef)
      (qemudAssignVMDef, qemudSaveVMDef, qemudSaveNetworkConfig)
      (qemudParseDhcpRangesXML, qemudParseNetworkXML)
      (qemudAssignNetworkDef, qemudSaveNetworkDef, qemudGenerateXML)
      (qemudGenerateNetworkXML, qemudDeleteConfig): Mark strings.
      * src/qemu_driver.c (qemudBuildDnsmasqArgv, qemudAddIptablesRules)
      (qemudGetCapabilities, qemudDomainGetOSType)
      (qemudListDefinedDomains, qemudListNetworks)
      (qemudListDefinedNetworks, qemudNetworkGetBridgeName): Mark strings.
      b53c4e51
    • J
      ReportError: mark for translation string args to this function · 3163224b
      Jim Meyering 提交于
      * Makefile.maint (msg_gen_function): Add ReportError.
      * src/util.c (_virExec): Mark strings.
      3163224b
    • J
      xenXMError: mark for translation string args to this function · 56a3cb80
      Jim Meyering 提交于
      * Makefile.maint (msg_gen_function): Add xenXMError.
      * src/xm_internal.c (xenXMParseXMLToConfig, xenXMDomainDefineXML):
      (xenXMDomainAttachDevice, xenXMDomainDetachDevice): Mark strings.
      56a3cb80
    • J
      Mark many more strings for translation. · 513bd04c
      Jim Meyering 提交于
      * Makefile.maint (err_func_re): Add to the list and make it readable.
      * po/POTFILES.in: Add src/util.c and src/xm_internal.c, and sort.
      * src/storage_backend.c: Fix comment.
      * src/util.c (virFileLinkPointsTo): Mark a string.
      * qemud/remote.c (remoteDispatchClientRequest): Mark strings.
      (remoteDispatchOpen, CHECK_CONN, remoteDispatchGetType): Likewise.
      (remoteDispatchDomainGetSchedulerType): Likewise.
      (remoteDispatchDomainGetSchedulerParameters): Likewise.
      (remoteDispatchDomainSetSchedulerParameters): Likewise.
      (remoteDispatchDomainBlockStats): Likewise.
      (remoteDispatchDomainInterfaceStats): Likewise.
      (remoteDispatchDomainAttachDevice, remoteDispatchDomainCreate):
      (remoteDispatchDomainDestroy, remoteDispatchDomainDetachDevice):
      (remoteDispatchDomainDumpXml, remoteDispatchDomainGetAutostart):
      (remoteDispatchDomainGetInfo, remoteDispatchDomainGetMaxMemory):
      (remoteDispatchDomainGetMaxVcpus, remoteDispatchDomainGetOsType):
      (remoteDispatchDomainGetVcpus): Likewise.
      (remoteDispatchDomainMigratePerform): Likewise.
      (remoteDispatchListDefinedDomains, remoteDispatchDomainPinVcpu):
      (remoteDispatchDomainReboot, remoteDispatchDomainResume):
      (remoteDispatchDomainSave, remoteDispatchDomainCoreDump):
      (remoteDispatchDomainSetAutostart): Likewise.
      (remoteDispatchDomainSetMaxMemory, remoteDispatchDomainSetMemory):
      (remoteDispatchDomainSetVcpus, remoteDispatchDomainShutdown):
      (remoteDispatchDomainSuspend, remoteDispatchDomainUndefine):
      (remoteDispatchListDefinedNetworks, remoteDispatchListDomains):
      (remoteDispatchListNetworks, remoteDispatchNetworkCreate):
      (remoteDispatchNetworkDestroy, remoteDispatchNetworkDumpXml):
      (remoteDispatchNetworkGetAutostart, remoteDispatchNetworkGetBridgeName):
      (remoteDispatchNetworkSetAutostart, remoteDispatchNetworkUndefine):
      (addrToString, remoteDispatchAuthSaslInit, remoteDispatchAuthSaslStart):
      (remoteDispatchAuthSaslStep, remoteDispatchListDefinedStoragePools):
      (remoteDispatchListStoragePools, remoteDispatchStoragePoolCreate):
      (remoteDispatchStoragePoolBuild, remoteDispatchStoragePoolDestroy):
      (remoteDispatchStoragePoolDelete, remoteDispatchStoragePoolRefresh):
      (remoteDispatchStoragePoolGetInfo, remoteDispatchStoragePoolDumpXml):
      (remoteDispatchStoragePoolGetAutostart): Likewise.
      (remoteDispatchStoragePoolSetAutostart):
      (remoteDispatchStoragePoolListVolumes):
      (remoteDispatchStoragePoolNumOfVolumes):
      (remoteDispatchStoragePoolUndefine, remoteDispatchStorageVolCreateXml):
      (remoteDispatchStorageVolDelete, remoteDispatchStorageVolGetInfo):
      (remoteDispatchStorageVolDumpXml, remoteDispatchStorageVolGetPath):
      (remoteDispatchStorageVolLookupByName): Likewise.
      * src/qemu_driver.c (qemudOpenMonitor, qemudStartVMDaemon):
      (dhcpStartDhcpDaemon, qemudStartNetworkDaemon):
      (qemudDomainSuspend, qemudDomainResume, qemudDomainShutdown):
      (qemudDomainGetOSType, qemudDomainSetMaxMemory):
      (qemudDomainSetMemory, qemudDomainGetInfo, qemudDomainSave):
      (qemudDomainRestore, qemudDomainDumpXML, qemudDomainStart):
      (qemudDomainUndefine, qemudDomainChangeCDROM):
      (qemudDomainAttachDevice, qemudDomainGetAutostart):
      (qemudDomainSetAutostart, qemudDomainInterfaceStats):
      (qemudNetworkLookupByUUID, qemudNetworkLookupByName):
      (qemudNetworkUndefine, qemudNetworkStart, qemudNetworkDestroy):
      (qemudNetworkDumpXML, qemudNetworkGetAutostart):
      (qemudNetworkSetAutostart): Likewise.
      * src/virsh.c (cmdVcpupin, cmdAttachDevice, cmdDetachDevice): Likewise.
      * src/xm_internal.c (xenXMConfigCacheRefresh, xenXMDomainPinVcpu): Likewise.
      513bd04c
  2. 24 3月, 2008 2 次提交
  3. 25 2月, 2008 1 次提交
    • J
      Avoid new "make syntax-check" failures. · e497deb8
      Jim Meyering 提交于
      * .x-sc_avoid_write: Exempt src/util-lib.c, too.
      * Makefile.maint (sc_unmarked_diagnostics): Filter out false positives.
      * src/storage_backend_fs.c (virStorageBackendFileSystemVolCreate):
      Use safewrite, not write.
      * src/storage_backend_logical.c (virStorageBackendLogicalBuildPool): Likewise.
      e497deb8
  4. 22 2月, 2008 1 次提交
    • J
      Use safewrite in place of write, in many cases. · b59d9c85
      Jim Meyering 提交于
      Also add "make syntax-check" rules to ensure no new uses sneak in.
      
      There are many uses of write like this:
      
          if (write (fd, xml, towrite) != towrite)
              return -1;
      
      The problem is that the syscall can succeed, yet write less than
      the requested number of bytes, so the caller should retry
      rather than simply failing.
      
      This patch changes most of them to use util.c's safewrite wrapper,
      which encapsulates the process.  Also, there were a few cases in
      which the retry loop was open-coded, and I replaced those, too.
      
      * Makefile.maint (sc_avoid_write): New rule, to avoid recurrence.
      * .x-sc_avoid_write: New file.  Record two legitimate exemptions.
      * qemud/qemud.c (sig_handler, qemudClientWriteBuf): Use safewrite, not write.
      * src/conf.c (__virConfWriteFile): Likewise.
      * src/qemu_conf.c (qemudSaveConfig, qemudSaveNetworkConfig): Likewise.
      * src/qemu_driver.c (qemudWaitForMonitor, qemudStartVMDaemon)
      (qemudVMData, PROC_IP_FORWARD): Likewise.
      * proxy/libvirt_proxy.c: Include "util.h".
      (proxyWriteClientSocket): Use safewrite.
      * src/test.c (testDomainSave, testDomainCoreDump): Likewise.
      * src/proxy_internal.c (virProxyWriteClientSocket): Likewise.
      * src/virsh.c: Include "util-lib.h".
      (vshOutputLogFile): Use safewrite.
      * src/console.c: Include "util-lib.h".
      (vshRunConsole): Use safewrite.
      b59d9c85
  5. 20 2月, 2008 1 次提交
  6. 08 2月, 2008 3 次提交
    • J
      Enable another syntax-check rule. · db4037e5
      Jim Meyering 提交于
      * Makefile.cfg (local-checks-to-skip): Remove sc_no_have_config_h.
      * qemud/mdns.c: Remove "#ifdef HAVE_CONFIG_H".
      * Makefile.maint (sc_no_have_config_h): Tighten up regexp,
      so that the above mention of "HAVE_CONFIG_H" doesn't match.
      * .x-sc_no_have_config_h: New file, to exempt gnulib/ straggler.
      db4037e5
    • J
      Mark all qemudLog diagnostics for translation. · 247df6e4
      Jim Meyering 提交于
      * po/POTFILES.in: Add names of many new files.
      * Makefile.maint (err_func_re): Add qemudLog.
      Mark diagnostics with _(...).  Split some long lines.
      * qemud/qemud.c (remoteCheckCertFile, remoteInitializeGnuTLS):
      (qemudDispatchSignalEvent, qemudSetCloseExec, qemudSetNonBlock):
      (qemudWritePidFile, qemudListenUnix, remoteMakeSockets):
      (remoteListenTCP, qemudInitPaths, qemudInitialize):
      (qemudNetworkInit, remoteInitializeTLSSession, remoteCheckDN):
      (remoteCheckCertificate, remoteCheckAccess, qemudDispatchServer):
      (qemudClientReadBuf, qemudDispatchClientRead):
      (qemudClientWriteBuf, qemudDispatchClientWrite, qemudOneLoop):
      (remoteConfigGetStringList, checkType, GET_CONF_STR):
      (remoteConfigGetAuth, remoteReadConfigFile, main):
      * qemud/remote.c (remoteDispatchAuthSaslInit, remoteSASLCheckSSF):
      (remoteSASLCheckAccess, remoteDispatchAuthSaslStart):
      (remoteDispatchAuthSaslStep, remoteDispatchAuthSaslInit):
      (remoteDispatchAuthSaslStart, remoteDispatchAuthSaslStep):
      (qemudGetSocketIdentity, remoteDispatchAuthPolkit):
      * src/iptables.c (notifyRulesUpdated, MAX_FILE_LEN, iptRulesSave):
      (iptRulesReload):
      * src/qemu_conf.c (qemudExtractVersionInfo, qemudLoadConfig):
      (qemudLoadNetworkConfig, qemudScanConfigDir):
      * src/qemu_driver.c (qemudSetCloseExec, qemudSetNonBlock):
      (qemudAutostartConfigs, qemudStartup, qemudReload):
      (qemudWaitForMonitor, qemudStartVMDaemon, qemudVMData):
      (qemudShutdownVMDaemon, qemudStartNetworkDaemon):
      (qemudShutdownNetworkDaemon, qemudMonitorCommand):
      (qemudDomainUndefine, qemudNetworkUndefine):
      * src/uuid.c (virUUIDGenerate):
      * src/xm_internal.c (xenXMAttachInterface):
      247df6e4
    • J
      Remove more useless if tests before "free"-like functions. · e8ff93b4
      Jim Meyering 提交于
      * build-aux/useless-if-before-free: Rename from ...
      * build-aux/find-unnecessary-if-before-free: ... this.  Remove file.
      Also changed it so that new names are no longer hard-coded in the
      script.  Instead, they're supplied via options:
      * Makefile.cfg (useless_free_options): Define.
      Add xmlXPathFreeObject to the list of free-like functions it detects.
      * Makefile.maint (sc_avoid_if_before_free): Reflect script renaming.
      * .x-sc_avoid_if_before_free: Likewise.
      * src/openvz_conf.c (openvzParseXML): Remove useless "if"-before-free.
      * src/qemu_conf.c (qemudParseXML, qemudParseNetworkXML): Likewise.
      * src/virsh.c (cmdVNCDisplay, cmdTTYConsole, cmdDetachInterface):
      (cmdDetachDisk): Likewise.
      * src/xm_internal.c (xenXMConfigSetIntFromXPath): Likewise.
      (xenXMConfigSetStringFromXPath, xenXMParseXMLToConfig): Likewise.
      (xenXMDomainAttachDevice, xenXMAttachDisk, xenXMAttachInterface):
      (xenXMDomainDetachDevice): Likewise.
      * src/xml.c (virXPathString): Likewise.
      * tests/xmlrpctest.c (checkRequestValue): Likewise.
      e8ff93b4
  7. 05 2月, 2008 2 次提交
    • J
      Eliminate all uses of virBufferAdd with string literals. · fb66d5ab
      Jim Meyering 提交于
      * Makefile.maint (sc_prohibit_virBufferAdd_with_string_literal):
      New rule.
      * src/buf.h (virBufferAddLit): Define.
      * src/conf.c (virConfSaveValue): Use virBufferAddLit, in place
      of virBufferAdd everywhere possible.
      (virConfSaveEntry): Likewise.
      * src/qemu_conf.c (qemudGenerateXML, qemudGenerateNetworkXML): Likewise.
      * src/qemu_driver.c (qemudGetFeatures, qemudGetCapabilities): Likewise.
      * src/test.c (testDomainDumpXML, testNetworkDumpXML): Likewise.
      * src/xen_internal.c (xenHypervisorMakeCapabilitiesXML): Likewise.
      * src/xend_internal.c (xend_parse_sexp_desc_os): Likewise.
      (xend_parse_sexp_desc, sexpr_to_xend_topology_xml): Likewise.
      * src/xm_internal.c (xenXMDomainFormatXML, xenXMDomainPinVcpu): Likewise.
      * src/xml.c (virSaveCpuSet, virParseXenCpuTopology): Likewise.
      (virDomainParseXMLGraphicsDescImage): Likewise.
      (virDomainParseXMLGraphicsDescVFB, virDomainParseXMLOSDescHVM): Likewise.
      (virDomainParseXMLOSDescPV, virDomainParseXMLDiskDesc): Likewise.
      (virDomainParseXMLIfDesc, virDomainParseXMLDesc): Likewise.
      fb66d5ab
    • J
      Enable more checks for unmarked messages; fix violators. · ecaa6962
      Jim Meyering 提交于
      * Makefile.maint (sc_unmarked_diagnostics): Check for more functions.
      * src/conf.c (virConfParseValue): Remove trailing space in diagnostic.
      * src/libvirt.c (do_open, virDomainMigrate): Mark diagnostics.
      * src/xend_internal.c (do_connect, xenDaemonDomainDumpXMLByID): Likewise.
      (xenDaemonDomainDumpXMLByName, xenDaemonCreateLinux): Likewise.
      (xenDaemonDomainMigratePerform, xenDaemonDomainDefineXML): Likewise.
      * src/xml.c (virXPathString, virXPathNumber, virXPathLong): Likewise.
      (virXPathBoolean, virXPathNode, virXPathNodeSet): Likewise.
      (virDomainParseXMLOSDescHVM): Likewise.
      * src/xmlrpc.c (xmlRpcValueNew): Likewise.
      (xmlRpcValueUnmarshalDictElement): Likewise.
      ecaa6962
  8. 30 1月, 2008 1 次提交
    • J
      Add framework for code style- and syntax-checking rules. · cc337da2
      Jim Meyering 提交于
      	Almost all tests are initially disabled via the list in Makefile.cfg.
      	* Makefile.am (EXTRA_DIST): Add .x-sc_avoid_if_before_free.
      	Omit names of files that automake includes automatically.
      	* .x-sc_avoid_if_before_free: New file.
      	* build-aux/vc-list-files: Likewise.
      	* build-aux/find-unnecessary-if-before-free: Likewise.
      	* GNUmakefile, Makefile.cfg, Makefile.maint: New files.
      cc337da2