Fix TestMoveNearest on Windows

[amccarth]

Summary

The header file for the DLL tried to declare inline functions and a local
function as dllexport which broke the compile and link. Removing the bad
declarations solves the problem, and the test passes on Windows now.

[zturner]

Do we not need call_foo2 anymore?

[amccarth]

In D43600#1016483, @zturner wrote:

Do we not need call_foo2 anymore?

call_foo2 is defined in the executable, so attempting to report it as dllexport from the DLL is nonsensical and causes link-time errors.

[zturner]

How is call_foo2 any different than call_foo1, which was not removed from this patch?

[amccarth]

call_foo1() is defined in foo.cpp, which is built into a DLL, and thus needs the __declspec(dllimport/dllexport) from the LLDB_TEST_API. foo.h defines the API for the DLL.

call_foo2() is defined in main.cpp, which is built into an executable that depends on foo.dll.

[zturner]

Ok, that's the piece that was missing since the code for foo.cpp and main.cpp weren't part of the patch.

lldb/trunk/packages/Python/lldbsuite/test/functionalities/breakpoint/move_nearest/foo.h

-LLDB_TEST_API inline int foo1() { return 1; } // !BR1
+inline int foo1() { return 1; } // !BR1
 
-LLDB_TEST_API inline int foo2() { return 2; } // !BR2
+inline int foo2() { return 2; } // !BR2
 
 LLDB_TEST_API extern int call_foo1();
-LLDB_TEST_API extern int call_foo2();