hari-om-888 opened a new pull request #84:
URL: https://github.com/apache/commons-net/pull/84


   `SubnetUtils` classes is right now initialised with `inclusiveHostCount`  as 
false by default.
   Generally we are either interested into inclusiveHostCount being false or 
true since the inception of use case. We do not change it after the object of 
`SubnetUtils` have been created.
   
   Also if `inclusiveHostCount` is supported as constructor parameters, it 
makes the use case of `inclusiveHostCount` more verbose. Otherwise it is very 
difficult for new person to find out what if such a parameter exist let alone 
default behaviour of it being false.
   
   This pr adds constructors which support `inclusiveHostCount` as constructor 
parameters.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to