Resolves: tdf#88953 byref args incorrectly passed in basic->c++

partial revert

of

commit 22b094f5
Date:   Tue Jul 14 14:50:07 2015 +0200

    loplugin:unusedmethods basic

and partial revert of...

commit e2080e70
Date:   Wed Jun 18 12:14:29 2014 +0200

    new compilerplugin returnbyref

and revert of...

commit 536051f8
Date:   Wed Jul 15 14:04:01 2015 +0200

    fix Windows build

    after commit 22b094f5
    "loplugin:unusedmethods basic"

Change-Id: I612937334fdb75365080c98a9d4da5ed7ae647e3

basic/source/runtime/dllmgr-x64.cxx

@@ -315,7 +315,7 @@ SbError marshal(
             case SbxDOUBLE:
             case SbxBOOL:
             case SbxBYTE:
-                add(blob, variable->GetValues_Impl(), 8, offset);
+                add(blob, variable->data(), 8, offset);
                 break;
             case SbxSTRING:
                 {

basic/source/runtime/dllmgr-x86.cxx

@@ -330,7 +330,7 @@ SbError marshal(
             case SbxDOUBLE:
             case SbxBOOL:
             case SbxBYTE:
-                add(blob, variable->GetValues_Impl(), 4, offset);
+                add(blob, variable->data(), 4, offset);
                 break;
             case SbxSTRING:
                 {

include/basic/sbxvar.hxx

@@ -119,6 +119,8 @@ public:
     const SbxValues& GetValues_Impl() const { return aData; }
     bool Put( const SbxValues& );
 
+    SbxValues * data() { return &aData; }
+
     sal_Unicode GetChar() const;
     sal_Int16   GetInteger() const;
     sal_Int32   GetLong() const;