[jira] [Created] (HDFS-10554) libhdfs++: signed to unsigned conversions are breaking things and compiler isn't issuing expected warnings

["James Clampffer (JIRA)" <ji...@apache.org>]

James Clampffer created HDFS-10554:
--------------------------------------

             Summary: libhdfs++: signed to unsigned conversions are breaking things and compiler isn't issuing expected warnings
                 Key: HDFS-10554
                 URL: https://issues.apache.org/jira/browse/HDFS-10554
             Project: Hadoop HDFS
          Issue Type: Sub-task
            Reporter: James Clampffer
            Assignee: James Clampffer


There's at least two places where we use -1 to indicate unset/default values that end up getting cast into unsigned integers.  The compiler should be smart enough to figure this out and issue a warning but it's not; we need to find out what's going on there.  We also need to fix the places where this sort of thing has found its way into the code:

In URI
{code}
      // -1 if the port is undefined.
      optional<uint16_t> get_port() const
      { return port; }
{code}

In Options (gets cast to uint64_t somewhere)
{code}
    /**
     * Maximum number of retries for RPC operations
     **/
    int max_rpc_retries;
    static const int kNoRetry = -1;
{code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-dev-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-help@hadoop.apache.org