andreacavalli/rocksdb
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
40cd91b7fe
rocksdb/java/rocksjni
History
Yueh-Hsuan Chiang 40cd91b7fe Fixed compile warning in rocksdbjava 
Summary:
Fixed the following compile warning in rocksdbjava:

    java/rocksjni/comparatorjnicallback.cc:20:14: warning: unused variable ‘rs’ [-Wunused-variable]
       const jint rs = env->GetJavaVM(&m_jvm);
                  ^
    java/rocksjni/comparatorjnicallback.cc: In member function ‘JNIEnv* rocksdb::BaseComparatorJniCallback::getJniEnv() const’:
    java/rocksjni/comparatorjnicallback.cc:45:8: warning: unused variable ‘rs’ [-Wunused-variable]
       jint rs = m_jvm->AttachCurrentThread(reinterpret_cast<void **>(&env), NULL);
            ^
    java/rocksjni/loggerjnicallback.cc: In constructor ‘rocksdb::LoggerJniCallback::LoggerJniCallback(JNIEnv*, jobject)’:
    java/rocksjni/loggerjnicallback.cc:19:14: warning: unused variable ‘rs’ [-Wunused-variable]
       const jint rs = env->GetJavaVM(&m_jvm);
                  ^
    java/rocksjni/loggerjnicallback.cc: In member function ‘JNIEnv* rocksdb::LoggerJniCallback::getJniEnv() const’:
    java/rocksjni/loggerjnicallback.cc:33:8: warning: unused variable ‘rs’ [-Wunused-variable]
       jint rs = m_jvm->AttachCurrentThread(reinterpret_cast<void **>(&env), NULL);
            ^

Test Plan: make rocksdbjava -j32

Reviewers: sdong, anthony, IslamAbdelRahman, igor

Reviewed By: igor

Subscribers: dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D45891
2015-08-31 18:36:17 -07:00
..
backupablejni.cc [RocksJava] Fixed MacOS build of RocksJava 2014-11-26 20:53:23 +01:00
backupenginejni.cc Expose the BackupEngine from the Java API 2015-08-04 16:08:54 -07:00
checkpoint.cc [RocksJava] Addressed comments in D28971 2014-11-25 23:33:42 +01:00
columnfamilyhandle.cc [RocksJava] Column family support 2014-10-13 10:34:52 +02:00
compaction_filter.cc The ability to specify a compaction filter via the Java API 2015-07-16 11:50:10 +01:00
comparator.cc Fix code review comments raised in https://reviews.facebook.net/D22779 2014-10-21 15:52:28 +01:00
comparatorjnicallback.cc Fixed compile warning in rocksdbjava 2015-08-31 18:36:17 -07:00
comparatorjnicallback.h Fix code review comments raised in https://reviews.facebook.net/D22779 2014-10-21 15:52:28 +01:00
env.cc [RocksJava] Expose MemEnv in RocksJava 2015-03-23 18:59:31 +01:00
filter.cc Filters getting disposed by System.gc before EOL 2014-10-15 20:59:25 +02:00
iterator.cc Abstractions for common iterator behaviour 2015-01-14 21:16:06 +00:00
loggerjnicallback.cc Fixed compile warning in rocksdbjava 2015-08-31 18:36:17 -07:00
loggerjnicallback.h RocksJava - JNI Logger callback 2015-03-14 20:57:18 +01:00
memtablejni.cc [RocksJava] 32-Bit adjustments 2015-03-19 20:45:55 +01:00
merge_operator.cc Integrated feedback from ankgup87 2014-10-13 15:37:29 -07:00
options.cc Fixed RocksJava test failure of shouldSetTestCappedPrefixExtractor 2015-08-04 17:01:08 -07:00
portal.h Expose the BackupEngine from the Java API 2015-08-04 16:08:54 -07:00
ratelimiterjni.cc 32-Bit RocksJava resolution for jlong overflows 2014-10-12 22:42:18 +02:00
remove_emptyvalue_compactionfilterjni.cc Java facility to use the RemoveEmptyValueCompactionFilter 2015-07-16 11:50:11 +01:00
restorejni.cc [RocksJava] Fixed MacOS build of RocksJava 2014-11-26 20:53:23 +01:00
rocksjni.cc Use CompactRangeOptions for CompactRange 2015-06-17 14:36:14 -07:00
slice.cc [RocksJava] Integrated proposed simplificiation 2015-01-17 01:22:29 +01:00
snapshot.cc [RocksJava] Snapshot - GetSequenceNumber 2015-01-31 15:50:34 +01:00
statistics.cc Adding iterator JNI binding 2014-04-19 03:26:22 -07:00
table.cc [RocksJava] BlockBasedTableConfig 3.10 2015-01-22 23:46:38 +01:00
transaction_log.cc [RocksJava] Incorporated changes for D32151 2015-01-31 15:15:49 +01:00
ttl.cc [RocksJava] CF Name shall handle bytes correctly 2015-02-10 21:00:00 +01:00
write_batch_test.cc Optimistic Transactions 2015-05-29 14:36:35 -07:00
write_batch_with_index.cc [RocksJava] Fixed test failures 2015-07-01 23:22:03 -07:00
write_batch.cc Moved Java test classes into src/test/java 2015-02-01 19:30:31 +00:00
writebatchhandlerjnicallback.cc Fix iOS compile with -Wshorten-64-to-32 2014-11-13 14:39:30 -05:00
writebatchhandlerjnicallback.h Addresed comments from code review https://reviews.facebook.net/D27567 2014-11-09 16:10:11 +00:00