DaddyFrosty/yoga
1
0
Fork 0
Code Issues 96 Pull Requests 31 Actions Packages Projects Releases 33 Wiki Activity

Removes unnecessary CSSLayoutContext param from CalculateLayout

Browse Source 
Summary:
Closes #222 for the `csharp` implementation which does not use the `CSSLayoutContext` parameter in the `CalculateLayout` method call.
Closes https://github.com/facebook/css-layout/pull/223

Reviewed By: lucasr

Differential Revision: D3918948

Pulled By: emilsjolander

fbshipit-source-id: 8708e08b226500727e33d131ed880bb7e077c08e
This commit is contained in:
mattpodwysocki
2016-09-27 04:34:52 -07:00
committed by Facebook Github Bot 0
parent 15d5cb0169
commit 9c93c7fc42
2 changed files with 1 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
csharp/Facebook.CSSLayout/CSSLayoutContext.cs
View File

@@ -1,17 +0,0 @@
/**
* Copyright (c) 2014-present, Facebook, Inc.
* All rights reserved.
*
* This source code is licensed under the BSD-style license found in the
* LICENSE file in the root directory of this source tree. An additional grant
* of patent rights can be found in the PATENTS file in the same directory.
*/
namespace Facebook.CSSLayout
{
public class CSSLayoutContext
{
public MeasureOutput measureOutput { get; } = new MeasureOutput();
public int CurrentGenerationCount { get; set; }
}
}

2
csharp/Facebook.CSSLayout/CSSNode.cs
View File

@@ -628,7 +628,7 @@ namespace Facebook.CSSLayout
Native.CSSNodeSetMeasureFunc(_cssNode, measureFunction != null ? _measureFunc : null); Native.CSSNodeSetMeasureFunc(_cssNode, measureFunction != null ? _measureFunc : null);
} }
public void CalculateLayout(CSSLayoutContext layoutContext) public void CalculateLayout()
{ {
CheckDisposed(); CheckDisposed();
Native.CSSNodeCalculateLayout(_cssNode, CSSConstants.Undefined, CSSConstants.Undefined, Native.CSSNodeStyleGetDirection(_cssNode)); Native.CSSNodeCalculateLayout(_cssNode, CSSConstants.Undefined, CSSConstants.Undefined, Native.CSSNodeStyleGetDirection(_cssNode));