Add fnl build target

mojo/public/cpp/system/tests/macros_unittest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file tests the C++ Mojo system macros and consists of "positive" tests,
 // i.e., those verifying that things work (without compile errors, or even
 // warnings if warnings are treated as errors).
 // TODO(vtl): Maybe rename "MacrosCppTest" -> "MacrosTest" if/when this gets
 // compiled into a different binary from the C API tests.
 // TODO(vtl): Fix no-compile tests (which are all disabled; crbug.com/105388)
@@ skipping 66 matching lines @@
 const int kGlobalArray[5] = {1, 2, 3, 4, 5};
 static_assert(MOJO_ARRAYSIZE(kGlobalArray) == 5u,
               "MOJO_ARRAY_SIZE() failed in static_assert()");
 
 TEST(MacrosCppTest, ArraySize) {
   double local_array[4] = {6.7, 7.8, 8.9, 9.0};
   // MSVS considers this local variable unused since MOJO_ARRAYSIZE only takes
   // the size of the type of the local and not the values itself.
   MOJO_ALLOW_UNUSED_LOCAL(local_array);
   EXPECT_EQ(4u, MOJO_ARRAYSIZE(local_array));
+
+  // Prevent gcc unneeded-internal-declaration warning

[viettrungluu, 2015/09/15 23:48:22]

nit: period at the end of the comment.

[cdotstout, 2015/09/16 22:01:00]

Done.

+  MOJO_ALLOW_UNUSED_LOCAL(kGlobalArray);

[viettrungluu, 2015/09/15 23:48:22]

"LOCAL". Sigh. <shrug>

[cdotstout, 2015/09/16 22:01:00]

Acknowledged.

 }
 
 // Note: MSVS is very strict (and arguably buggy) about warnings for classes
 // defined in a local scope, so define these globally.
 class MoveOnlyInt {
   MOJO_MOVE_ONLY_TYPE(MoveOnlyInt)
 
  public:
   MoveOnlyInt() : is_set_(false), value_() {}
   explicit MoveOnlyInt(int value) : is_set_(true), value_(value) {}
@@ skipping 53 matching lines @@
 int ReturnsIntYouMustUse() {
   return 123;
 }
 
 TEST(MacrosCppTest, IgnoreResult) {
   ignore_result(ReturnsIntYouMustUse());
 }
 
 }  // namespace
 }  // namespace mojo