Added ref modifier on SciterValue to window variable methods

Author: trueromanus

src/EmptyFlow.SciterAPI/Client/HostValueAPI.cs

@@ -229,7 +229,7 @@ public SciterValue ValueInvoke ( ref SciterValue value, SciterValue? context, IE
 		/// </summary>
 		/// <returns>Return element.</returns>
 		/// <exception cref="Exception">If Result will be not zero it will be part of error message.</exception>
-		public nint ElementFromValue ( SciterValue sciterValue ) {
+		public nint ElementFromValue ( ref SciterValue sciterValue ) {
 			var pointer = nint.Zero;
 			var domResult = m_basicApi.SciterElementUnwrap ( sciterValue, ref pointer );
 			if ( domResult != 0 ) throw new Exception ( $"Can't get element from value. DomResult is {domResult}" );
@@ -255,15 +255,15 @@ public SciterValue ElementToValue ( nint element ) {
 		/// <summary>
 		/// Sets variable that will be available in each document loaded after this call.
 		/// </summary>
-		public void SetSharedVariable ( string name, SciterValue value ) {
+		public void SetSharedVariable ( string name, ref SciterValue value ) {
 			var code = m_basicApi.SciterSetVariable ( nint.Zero, name, value );
 			if ( code != (uint) DomResult.SCDOM_OK ) throw new Exception ( $"Can't set variable {name}. Error is {code}." );
 		}
 
 		/// <summary>
 		/// Sets variable that will be available in root document of main window, call it in or after DOCUMENT_CREATED event.
 		/// </summary>
-		public void SetMainWindowVariable ( string name, SciterValue value ) {
+		public void SetMainWindowVariable ( string name, ref SciterValue value ) {
 			var code = m_basicApi.SciterSetVariable ( m_mainWindow, name, value );
 			if ( code != (uint) DomResult.SCDOM_OK ) throw new Exception ( $"Can't set variable {name}. Error is {code}." );
 		}

src/EmptyFlow.SciterAPI/Client/HostWindowsAPI.cs

@@ -362,7 +362,7 @@ public nint GetWindowFocus ( nint window ) {
 			if ( m_basicApi.SciterEval ( window, script, (uint) script.Length, out var result ) ) {
 				if ( result.IsNull ) return nint.Zero;
 
-				return ElementFromValue ( result );
+				return ElementFromValue ( ref result );
 			}
 
 			return nint.Zero;
@@ -373,7 +373,7 @@ public nint GetWindowParent ( nint window ) {
 			if ( m_basicApi.SciterEval ( window, script, (uint) script.Length, out var result ) ) {
 				if ( result.IsNull ) return nint.Zero;
 
-				return ElementFromValue ( result );
+				return ElementFromValue ( ref result );
 			}
 
 			return nint.Zero;

src/EmptyFlow.SciterAPI/EmptyFlow.SciterAPI.csproj

@@ -21,14 +21,9 @@
 		<Version>1.2.4.0</Version>
 		<PackageLicenseFile>LICENSE</PackageLicenseFile>
 		<PackageReleaseNotes>
-			IPseudoSomModel.SetUnique for set model unique identifier
-			IPseudoSomModel.Unique for get model unique identifier
-			SciterEventHandler.SetReturnState for some handlers it required to change return state
-			WindowEventHandler now obsolete, it redundant class please use SciterEventHandler instead in all cases
-			SciterAPIHost.AddWindowEventHandler added argument behaviours for change tracked behaviours
-			SciterAPIHost.UpdateElement for (re)render element
-			SciterAPIHost.ScrollToElement scroll to element
-			Fixed issue in showing different from main windows
+			SciterAPIHost.ElementFromValue first argument now have ref modificator
+			SciterAPIHost.SetSharedVariable argument value now have ref modificator
+			SciterAPIHost.SetMainWindowVariable argument value now have ref modificator
 		</PackageReleaseNotes>
 		<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
 	</PropertyGroup>