Hello community,

here is the log from the commit of package yast2-ycp-ui-bindings for 
openSUSE:Factory checked in at 2017-11-19 11:06:55
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-ycp-ui-bindings (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-ycp-ui-bindings.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-ycp-ui-bindings"

Sun Nov 19 11:06:55 2017 rev:71 rq:542098 version:4.0.0

Changes:
--------
--- 
/work/SRC/openSUSE:Factory/yast2-ycp-ui-bindings/yast2-ycp-ui-bindings.changes  
    2017-04-12 17:10:24.276623052 +0200
+++ 
/work/SRC/openSUSE:Factory/.yast2-ycp-ui-bindings.new/yast2-ycp-ui-bindings.changes
 2017-11-19 11:07:00.844963399 +0100
@@ -1,0 +2,7 @@
+Wed Nov 15 17:31:59 CET 2017 - [email protected]
+
+- Fixed segfault when comparing YCPValueWidgetIDs with
+  YStringWidgetIDs (fate#324098)
+- 4.0.0
+
+-------------------------------------------------------------------

Old:
----
  yast2-ycp-ui-bindings-3.2.0.tar.bz2

New:
----
  yast2-ycp-ui-bindings-4.0.0.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-ycp-ui-bindings.spec ++++++
--- /var/tmp/diff_new_pack.8LoiSb/_old  2017-11-19 11:07:01.336945459 +0100
+++ /var/tmp/diff_new_pack.8LoiSb/_new  2017-11-19 11:07:01.340945314 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-ycp-ui-bindings
-Version:        3.2.0
+Version:        4.0.0
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

++++++ yast2-ycp-ui-bindings-3.2.0.tar.bz2 -> 
yast2-ycp-ui-bindings-4.0.0.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-ycp-ui-bindings-3.2.0/README.md 
new/yast2-ycp-ui-bindings-4.0.0/README.md
--- old/yast2-ycp-ui-bindings-3.2.0/README.md   2017-04-10 14:06:12.716990458 
+0200
+++ new/yast2-ycp-ui-bindings-4.0.0/README.md   2017-11-15 18:30:27.429202615 
+0100
@@ -3,4 +3,6 @@
 Travis:  [![Build 
Status](https://travis-ci.org/yast/yast-ycp-ui-bindings.svg?branch=master)](https://travis-ci.org/yast/yast-ycp-ui-bindings)
 Jenkins: [![Jenkins 
Build](http://img.shields.io/jenkins/s/https/ci.opensuse.org/yast-ycp-ui-bindings-master.svg)](https://ci.opensuse.org/view/Yast/job/yast-ycp-ui-bindings-master/)
 
+### Online Documentation
 
+The latest documentation is generated online 
[here](https://yast-ui-bindings.surge.sh/).
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-ycp-ui-bindings-3.2.0/package/yast2-ycp-ui-bindings.changes 
new/yast2-ycp-ui-bindings-4.0.0/package/yast2-ycp-ui-bindings.changes
--- old/yast2-ycp-ui-bindings-3.2.0/package/yast2-ycp-ui-bindings.changes       
2017-04-10 14:06:12.852990458 +0200
+++ new/yast2-ycp-ui-bindings-4.0.0/package/yast2-ycp-ui-bindings.changes       
2017-11-15 18:30:27.545202615 +0100
@@ -1,4 +1,11 @@
 -------------------------------------------------------------------
+Wed Nov 15 17:31:59 CET 2017 - [email protected]
+
+- Fixed segfault when comparing YCPValueWidgetIDs with
+  YStringWidgetIDs (fate#324098)
+- 4.0.0
+
+-------------------------------------------------------------------
 Mon Apr 10 10:31:28 UTC 2017 - [email protected]
 
 - added built-in SetApplicationTitle for setting application title
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-ycp-ui-bindings-3.2.0/package/yast2-ycp-ui-bindings.spec 
new/yast2-ycp-ui-bindings-4.0.0/package/yast2-ycp-ui-bindings.spec
--- old/yast2-ycp-ui-bindings-3.2.0/package/yast2-ycp-ui-bindings.spec  
2017-04-10 14:06:12.852990458 +0200
+++ new/yast2-ycp-ui-bindings-4.0.0/package/yast2-ycp-ui-bindings.spec  
2017-11-15 18:30:27.545202615 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-ycp-ui-bindings
-Version:        3.2.0
+Version:        4.0.0
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-ycp-ui-bindings-3.2.0/src/YCPValueWidgetID.cc 
new/yast2-ycp-ui-bindings-4.0.0/src/YCPValueWidgetID.cc
--- old/yast2-ycp-ui-bindings-3.2.0/src/YCPValueWidgetID.cc     2017-04-10 
14:06:12.864990458 +0200
+++ new/yast2-ycp-ui-bindings-4.0.0/src/YCPValueWidgetID.cc     2017-11-15 
18:30:27.561202615 +0100
@@ -44,7 +44,7 @@
     YCPValueWidgetID * otherYCPValueID =
        dynamic_cast<YCPValueWidgetID *> (otherID);
 
-    if ( otherID )
+    if ( otherYCPValueID )
        return _value->equal( otherYCPValueID->value() );
 
     if ( _value->isString() || _value->isSymbol() )
@@ -52,14 +52,17 @@
        YStringWidgetID * otherStringID =
            dynamic_cast<YStringWidgetID *> (otherID);
 
-       if ( _value->isString() )
-       {
-           return _value->asString()->value() == otherStringID->value();
-       }
-       else // _value->isSymbol()
-       {
-           return _value->asSymbol()->symbol() == otherStringID->value();
-       }
+        if ( otherStringID )
+        {
+            if ( _value->isString() )
+            {
+                return _value->asString()->value() == otherStringID->value();
+            }
+            else // _value->isSymbol()
+            {
+                return _value->asSymbol()->symbol() == otherStringID->value();
+            }
+        }
     }
 
     return false;
@@ -69,6 +72,9 @@
 string
 YCPValueWidgetID::toString() const
 {
+    // If value is Symbol - get symbol
+    if ( _value->isSymbol() ) return _value->asSymbol()->symbol();
+    // Else return as string
     return _value->toString();
 }
 
@@ -78,4 +84,3 @@
 {
     return _value;
 }
-


Reply via email to