Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(279)

Side by Side Diff: base/android/java/templates/NativeLibraries.template

Issue 793203002: Use SuppressFBWarnings to suppress findbugs warnings (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase + landmine Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.base.library_loader; 5 package org.chromium.base.library_loader;
6 6
7 import org.chromium.base.annotations.SuppressFBWarnings;
8
9 @SuppressFBWarnings
7 public class NativeLibraries { 10 public class NativeLibraries {
8 /** 11 /**
9 * IMPORTANT NOTE: The variables defined here must _not_ be 'final'. 12 * IMPORTANT NOTE: The variables defined here must _not_ be 'final'.
10 * 13 *
11 * The reason for this is very subtle: 14 * The reason for this is very subtle:
12 * 15 *
13 * - This template is used to generate several distinct, but similar 16 * - This template is used to generate several distinct, but similar
14 * files used in different contexts: 17 * files used in different contexts:
15 * 18 *
16 * o .../gen/templates/org/chromium/base/library_loader/NativeLibraries.ja va 19 * o .../gen/templates/org/chromium/base/library_loader/NativeLibraries.ja va
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 // This is the expected version of the 'main' native library, which is the o ne that 86 // This is the expected version of the 'main' native library, which is the o ne that
84 // implements the initial set of base JNI functions including 87 // implements the initial set of base JNI functions including
85 // base::android::nativeGetVersionName() 88 // base::android::nativeGetVersionName()
86 static String sVersionNumber = 89 static String sVersionNumber =
87 #if defined(NATIVE_LIBRARIES_VERSION_NUMBER) 90 #if defined(NATIVE_LIBRARIES_VERSION_NUMBER)
88 NATIVE_LIBRARIES_VERSION_NUMBER; 91 NATIVE_LIBRARIES_VERSION_NUMBER;
89 #else 92 #else
90 ""; 93 "";
91 #endif 94 #endif
92 } 95 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698