[PATCH 3/3] mlang/tests: Add MapFont tests.

[Ziqing Hui]

Signed-off-by: Ziqing Hui 
---
  dlls/mlang/tests/Makefile.in |  2 +-
  dlls/mlang/tests/mlang.c     | 99 
+++++++++++++++++++++++++++++++++++++++++++-
  2 files changed, 99 insertions(+), 2 deletions(-)

[Nikolay Sivov]

On 11/27/2017 02:47 PM, Ziqing Hui wrote:
> +static void test_MapFont(IMLangFontLink *font_link, IMLangFontLink2 *font_link2)

I know it's already done this way for other mlang tests, but I think 
it's better to have each test create its own objects, instead of reusing 
them. Creating IMLangFontLink and IMLangFontLink2 separately is also 
unnecessary, because you can query one from another.

[Huw Davies]

On Mon, Nov 27, 2017 at 03:06:53PM +0300, Nikolay Sivov wrote:
> On 11/27/2017 02:47 PM, Ziqing Hui wrote:
> Creating IMLangFontLink and IMLangFontLink2 separately is also unnecessary,
> because you can query one from another.

Actually, in this test, two objects are required to test the scope of
the cache.

Huw.

[Huw Davies]

Signed-off-by: Huw Davies