Re: svn commit: r325016 - head/lib/libdevdctl

2017-10-27 Thread Ngie Cooper

> On Oct 26, 2017, at 10:56, Alan Somers  wrote:
> 
> Author: asomers
> Date: Thu Oct 26 17:56:34 2017
> New Revision: 325016
> URL: https://svnweb.freebsd.org/changeset/base/325016
> 
> Log:
>  Partially revert r325011: restore Guid's default constructor
> 
>  Reported by:ohartmann
>  MFC after:3 weeks
>  X-MFC-With:325011
>  Sponsored by:Spectra Logic Corp
> 
> Modified:
>  head/lib/libdevdctl/guid.h

Or you could satisfy both signatures via optional parameters ;)..
-Ngie
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


svn commit: r325016 - head/lib/libdevdctl

2017-10-26 Thread Alan Somers
Author: asomers
Date: Thu Oct 26 17:56:34 2017
New Revision: 325016
URL: https://svnweb.freebsd.org/changeset/base/325016

Log:
  Partially revert r325011: restore Guid's default constructor
  
  Reported by:  ohartmann
  MFC after:3 weeks
  X-MFC-With:   325011
  Sponsored by: Spectra Logic Corp

Modified:
  head/lib/libdevdctl/guid.h

Modified: head/lib/libdevdctl/guid.h
==
--- head/lib/libdevdctl/guid.h  Thu Oct 26 17:45:01 2017(r325015)
+++ head/lib/libdevdctl/guid.h  Thu Oct 26 17:56:34 2017(r325016)
@@ -62,7 +62,11 @@ class Guid
 {
 public:
/* Constructors */
+   /* Default constructor: an Invalid guid */
+   Guid();
+   /* Construct a guid from a provided integer */
Guid(uint64_t guid);
+   /* Construct a guid from a string in base 8, 10, or 16 */
Guid(const std::string &guid);
static Guid InvalidGuid();
 
@@ -88,6 +92,12 @@ class Guid
 };
 
 //- Guid Inline Public Methods 
+inline
+Guid::Guid()
+  : m_GUID(INVALID_GUID)
+{
+}
+
 inline
 Guid::Guid(uint64_t guid)
   : m_GUID(guid)
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"