summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--o3d/build/all.gyp1
-rw-r--r--o3d/build/nacl.gyp4
-rw-r--r--o3d/core/core.gyp119
-rw-r--r--o3d/core/win/d3d9/effect_d3d9.cc4
-rw-r--r--o3d/import/cross/memory_stream_test.cc6
-rw-r--r--o3d/import/import.gyp35
-rw-r--r--o3d/plugin/plugin.gyp18
-rw-r--r--o3d/plugin/win/main_win.cc2
-rw-r--r--o3d/serializer/serializer.gyp11
-rw-r--r--o3d/statsreport/statsreport.gyp53
-rw-r--r--o3d/tests/build.scons12
-rw-r--r--o3d/tests/common/win/testing_common.cc3
-rw-r--r--o3d/utils/utils.gyp21
13 files changed, 176 insertions, 113 deletions
diff --git a/o3d/build/all.gyp b/o3d/build/all.gyp
index 6ca0803..43ea2ae 100644
--- a/o3d/build/all.gyp
+++ b/o3d/build/all.gyp
@@ -23,6 +23,7 @@
'../converter/converter.gyp:*',
'../core/core.gyp:*',
'../import/import.gyp:*',
+ '../tests/tests.gyp:*',
'../plugin/idl/idl.gyp:*',
'../plugin/plugin.gyp:*',
'../serializer/serializer.gyp:*',
diff --git a/o3d/build/nacl.gyp b/o3d/build/nacl.gyp
index 99ed5ec..8b84e89 100644
--- a/o3d/build/nacl.gyp
+++ b/o3d/build/nacl.gyp
@@ -34,7 +34,7 @@
'dummy_file_that_never_gets_built_so_scons_always_runs',
],
'action': [
- 'python',
+ 'C:/Python24/python.exe',
'<@(_inputs)',
'--output="<(nacl_output_dir)"',
'--configuration="<(CONFIGURATION)"',
@@ -43,7 +43,7 @@
],
},
],
- 'direct_dependent_settings': {
+ 'all_dependent_settings': {
'include_dirs': [
'<(nacldir)',
],
diff --git a/o3d/core/core.gyp b/o3d/core/core.gyp
index 1f27990..949b9f5 100644
--- a/o3d/core/core.gyp
+++ b/o3d/core/core.gyp
@@ -35,6 +35,7 @@
'../../<(pngdir)/libpng.gyp:libpng',
'../../<(zlibdir)/zlib.gyp:zlib',
'../../skia/skia.gyp:skia',
+ '../build/nacl.gyp:build_nacl',
],
'sources': [
'cross/bitmap.cc',
@@ -268,63 +269,6 @@
],
},
{
- 'target_name': 'o3dCoreTest',
- 'type': 'static_library',
- 'dependencies': [
- ],
- 'sources': [
- 'cross/bitmap_test.cc',
- 'cross/bounding_box_test.cc',
- 'cross/buffer_test.cc',
- 'cross/class_manager_test.cc',
- 'cross/client_test.cc',
- 'cross/counter_test.cc',
- 'cross/curve_test.cc',
- 'cross/draw_element_test.cc',
- 'cross/draw_list_test.cc',
- 'cross/draw_pass_test.cc',
- 'cross/effect_test.cc',
- 'cross/element_test.cc',
- 'cross/features_test.cc',
- 'cross/field_test.cc',
- 'cross/float_n_test.cc',
- 'cross/function_test.cc',
- 'cross/material_test.cc',
- 'cross/math_utilities_test.cc',
- 'cross/matrix4_axis_rotation_test.cc',
- 'cross/matrix4_composition_test.cc',
- 'cross/matrix4_scale_test.cc',
- 'cross/matrix4_translation_test.cc',
-# TODO(gspencer): fix this test so it can be re-enabled.
-# 'cross/message_queue_test.cc',
- 'cross/object_base_test.cc',
- 'cross/pack_test.cc',
- 'cross/param_array_test.cc',
- 'cross/param_object_test.cc',
- 'cross/param_operation_test.cc',
- 'cross/param_test.cc',
- 'cross/performance_timer_test.cc',
- 'cross/primitive_test.cc',
- 'cross/ray_intersection_info_test.cc',
- 'cross/render_node_test.cc',
- 'cross/renderer_test.cc',
- 'cross/service_locator_test.cc',
- 'cross/shape_test.cc',
- 'cross/skin_test.cc',
- 'cross/smart_ptr_test.cc',
- 'cross/state_set_test.cc',
- 'cross/state_test.cc',
- 'cross/stream_bank_test.cc',
- 'cross/stream_test.cc',
- 'cross/transform_test.cc',
- 'cross/tree_traversal_test.cc',
- 'cross/vector_map_test.cc',
- 'cross/vertex_source_test.cc',
- 'cross/visitor_base_test.cc',
- 'cross/weak_ptr_test.cc',
- ],
- },
- {
'target_name': 'o3dCorePlatform',
'type': 'static_library',
'dependencies': [
@@ -395,5 +339,66 @@
],
],
},
+ {
+ 'target_name': 'o3dCoreTest',
+ 'type': 'none',
+ 'dependencies': [
+ 'o3dCore',
+ 'o3dCorePlatform',
+ ],
+ 'direct_dependent_settings': {
+ 'sources': [
+ 'cross/bitmap_test.cc',
+ 'cross/bounding_box_test.cc',
+ 'cross/buffer_test.cc',
+ 'cross/class_manager_test.cc',
+ 'cross/client_test.cc',
+ 'cross/counter_test.cc',
+ 'cross/curve_test.cc',
+ 'cross/draw_element_test.cc',
+ 'cross/draw_list_test.cc',
+ 'cross/draw_pass_test.cc',
+ 'cross/effect_test.cc',
+ 'cross/element_test.cc',
+ 'cross/features_test.cc',
+ 'cross/field_test.cc',
+ 'cross/float_n_test.cc',
+ 'cross/function_test.cc',
+ 'cross/material_test.cc',
+ 'cross/math_utilities_test.cc',
+ 'cross/matrix4_axis_rotation_test.cc',
+ 'cross/matrix4_composition_test.cc',
+ 'cross/matrix4_scale_test.cc',
+ 'cross/matrix4_translation_test.cc',
+ # TODO(gspencer): fix this test so it can be re-enabled.
+ # 'cross/message_queue_test.cc',
+ 'cross/object_base_test.cc',
+ 'cross/pack_test.cc',
+ 'cross/param_array_test.cc',
+ 'cross/param_object_test.cc',
+ 'cross/param_operation_test.cc',
+ 'cross/param_test.cc',
+ 'cross/performance_timer_test.cc',
+ 'cross/primitive_test.cc',
+ 'cross/ray_intersection_info_test.cc',
+ 'cross/render_node_test.cc',
+ 'cross/renderer_test.cc',
+ 'cross/service_locator_test.cc',
+ 'cross/shape_test.cc',
+ 'cross/skin_test.cc',
+ 'cross/smart_ptr_test.cc',
+ 'cross/state_set_test.cc',
+ 'cross/state_test.cc',
+ 'cross/stream_bank_test.cc',
+ 'cross/stream_test.cc',
+ 'cross/transform_test.cc',
+ 'cross/tree_traversal_test.cc',
+ 'cross/vector_map_test.cc',
+ 'cross/vertex_source_test.cc',
+ 'cross/visitor_base_test.cc',
+ 'cross/weak_ptr_test.cc',
+ ],
+ },
+ },
],
}
diff --git a/o3d/core/win/d3d9/effect_d3d9.cc b/o3d/core/win/d3d9/effect_d3d9.cc
index 2bac2d9..830f5a4 100644
--- a/o3d/core/win/d3d9/effect_d3d9.cc
+++ b/o3d/core/win/d3d9/effect_d3d9.cc
@@ -254,7 +254,7 @@ class EffectParamSamplerArrayHandlerD3D9 : public EffectParamHandlerD3D9 {
<< "number of params in ParamArray does not match number of params "
<< "needed by shader array";
} else {
- for (int i = 0; i < size; ++i) {
+ for (unsigned i = 0; i < size; ++i) {
SamplerUnitIndexArray& index_array = sampler_unit_index_arrays_[i];
Param* untyped_element = param->GetUntypedParam(i);
// TODO(gman): Make this check happen when building the param cache.
@@ -297,7 +297,7 @@ class EffectParamSamplerArrayHandlerD3D9 : public EffectParamHandlerD3D9 {
if (param) {
unsigned size = param->size();
if (size == sampler_unit_index_arrays_.size()) {
- for (int i = 0; i < size; ++i) {
+ for (unsigned i = 0; i < size; ++i) {
SamplerUnitIndexArray& index_array = sampler_unit_index_arrays_[i];
Param* untyped_element = param->GetUntypedParam(i);
// TODO(gman): Make this check happen when building the param cache.
diff --git a/o3d/import/cross/memory_stream_test.cc b/o3d/import/cross/memory_stream_test.cc
index a02fcca..77ae95c 100644
--- a/o3d/import/cross/memory_stream_test.cc
+++ b/o3d/import/cross/memory_stream_test.cc
@@ -233,7 +233,7 @@ TEST_F(MemoryStreamTest, EndianSanityFloat32) {
uint8 *p8 = reinterpret_cast<uint8*>(p);
MemoryWriteStream write_stream(p8, sizeof(int32) * 2);
- float value = 3.14159;
+ float value = 3.14159f;
write_stream.WriteLittleEndianFloat32(value);
write_stream.WriteBigEndianFloat32(value);
@@ -261,9 +261,9 @@ TEST_F(MemoryStreamTest, EndianSanityFloat32) {
TEST_F(MemoryStreamTest, Endian) {
const int16 kValue1 = 13243;
const int32 kValue2 = 2393043;
- const float kValue3 = -0.039483;
+ const float kValue3 = -0.039483f;
const int16 kValue4 = -3984;
- const float kValue5 = 1234.5678;
+ const float kValue5 = 1234.5678f;
const uint8 kValue6 = 5; // write a single byte to make things interesting
const int32 kValue7 = -3920393;
diff --git a/o3d/import/import.gyp b/o3d/import/import.gyp
index 4ecb9c64..41cd4c4 100644
--- a/o3d/import/import.gyp
+++ b/o3d/import/import.gyp
@@ -126,5 +126,40 @@
],
],
},
+ {
+ 'target_name': 'o3dImportTest',
+ 'type': 'none',
+ 'dependencies': [
+ 'o3dArchive',
+ 'o3dImport',
+ '../../<(antlrdir)/antlr.gyp:antlr3c',
+ '../../<(fcolladadir)/fcollada.gyp:fcollada',
+ '../../<(jpegdir)/libjpeg.gyp:libjpeg',
+ '../../<(pngdir)/libpng.gyp:libpng',
+ '../../<(zlibdir)/zlib.gyp:zlib',
+ ],
+ 'direct_dependent_settings': {
+ 'sources': [
+ 'cross/gz_compressor_test.cc',
+ 'cross/gz_decompressor_test.cc',
+ 'cross/memory_buffer_test.cc',
+ 'cross/memory_stream_test.cc',
+ 'cross/raw_data_test.cc',
+ 'cross/tar_generator_test.cc',
+ 'cross/tar_processor_test.cc',
+ 'cross/targz_generator_test.cc',
+ 'cross/targz_processor_test.cc',
+ ],
+ 'conditions' : [
+ ['renderer == "d3d9" and OS == "win"',
+ {
+ 'include_dirs': [
+ '$(DXSDK_DIR)/Include',
+ ],
+ }
+ ],
+ ],
+ },
+ },
],
}
diff --git a/o3d/plugin/plugin.gyp b/o3d/plugin/plugin.gyp
index 86c4695..3966d40 100644
--- a/o3d/plugin/plugin.gyp
+++ b/o3d/plugin/plugin.gyp
@@ -21,6 +21,7 @@
'O3D_PLUGIN_MIME_TYPE="<!(python version_info.py --mimetype)"',
],
},
+
'targets': [
{
'target_name': 'npo3dautoplugin',
@@ -108,16 +109,21 @@
'msvs_settings': {
'VCLinkerTool': {
'AdditionalDependencies': [
+ 'rpcrt4.lib',
+ ],
+ },
+ },
+ },
+ ],
+ ['OS == "win" and renderer == "d3d9"',
+ {
+ 'msvs_settings': {
+ 'VCLinkerTool': {
+ 'AdditionalDependencies': [
'"$(DXSDK_DIR)/Lib/x86/DxErr9.lib"',
'"$(DXSDK_DIR)/Lib/x86/d3dx9.lib"',
- '../../<(cgdir)/lib/cg.lib',
- '../../<(cgdir)/lib/cgGL.lib',
'd3d9.lib',
- 'rpcrt4.lib',
],
- # Set /SUBSYSTEM:CONSOLE for converter.exe, since
- # it is a console app.
- 'SubSystem': '1',
},
},
},
diff --git a/o3d/plugin/win/main_win.cc b/o3d/plugin/win/main_win.cc
index db11b17..90f8d44 100644
--- a/o3d/plugin/win/main_win.cc
+++ b/o3d/plugin/win/main_win.cc
@@ -47,7 +47,7 @@
#include "plugin/cross/plugin_logging.h"
#include "plugin/cross/out_of_memory.h"
#include "statsreport/metrics.h"
-#include "third_party/v8/include/v8.h"
+#include "v8/include/v8.h"
#include "breakpad/win/bluescreen_detector.h"
using glue::_o3d::PluginObject;
diff --git a/o3d/serializer/serializer.gyp b/o3d/serializer/serializer.gyp
index 906f77f..9953aaf 100644
--- a/o3d/serializer/serializer.gyp
+++ b/o3d/serializer/serializer.gyp
@@ -30,10 +30,15 @@
},
{
'target_name': 'o3dSerializerTest',
- 'type': 'static_library',
- 'sources': [
- 'cross/serializer_test.cc',
+ 'type': 'none',
+ 'dependencies': [
+ 'o3dSerializer',
],
+ 'direct_dependent_settings': {
+ 'sources': [
+ 'cross/serializer_test.cc',
+ ],
+ },
},
],
}
diff --git a/o3d/statsreport/statsreport.gyp b/o3d/statsreport/statsreport.gyp
index 2b045d6..99b97e6 100644
--- a/o3d/statsreport/statsreport.gyp
+++ b/o3d/statsreport/statsreport.gyp
@@ -84,33 +84,38 @@
},
{
'target_name': 'o3dStatsReportTest',
- 'type': 'static_library',
- 'sources': [
- 'aggregator-win32_unittest.cc',
- 'aggregator-win32_unittest.h',
- 'aggregator_unittest.cc',
- 'aggregator_unittest.h',
- 'common/highres_timer_unittest.cc',
- 'formatter_unittest.cc',
- 'metrics_unittest.cc',
- 'persistent_iterator-win32_unittest.cc',
+ 'type': 'none',
+ 'dependencies': [
+ 'o3dStatsReport',
],
- 'conditions': [
- ['OS != "win"',
- {
- 'sources/': [
- ['exclude', '(-win32)_unittest\.(cc|h)$'],
- ],
- },
+ 'direct_dependent_settings': {
+ 'sources': [
+ 'aggregator-win32_unittest.cc',
+ 'aggregator-win32_unittest.h',
+ 'aggregator_unittest.cc',
+ 'aggregator_unittest.h',
+ 'common/highres_timer_unittest.cc',
+ 'formatter_unittest.cc',
+ 'metrics_unittest.cc',
+ 'persistent_iterator-win32_unittest.cc',
],
- ['OS != "mac"',
- {
- 'sources/': [
- ['exclude', '(-mac)_unittest\.(cc|mm|h)$'],
- ],
- },
+ 'conditions': [
+ ['OS != "win"',
+ {
+ 'sources/': [
+ ['exclude', '(-win32)_unittest\.(cc|h)$'],
+ ],
+ },
+ ],
+ ['OS != "mac"',
+ {
+ 'sources/': [
+ ['exclude', '(-mac)_unittest\.(cc|mm|h)$'],
+ ],
+ },
+ ],
],
- ],
+ },
},
],
},
diff --git a/o3d/tests/build.scons b/o3d/tests/build.scons
index 83efd83..218fbda 100644
--- a/o3d/tests/build.scons
+++ b/o3d/tests/build.scons
@@ -153,12 +153,12 @@ if env.Bit('windows'):
env.Help('\n')
swiftshader_install = []
-if env.Bit('windows'):
- # Copy SwiftShader to plugin dir.
- swiftshader_path = env.subst('$SWIFTSHADER_DIR/swiftshader_d3d9.dll')
- if os.path.exists(swiftshader_path):
- swiftshader_install = env.Replicate('$ARTIFACTS_DIR/O3DExtras',
- swiftshader_path)
+if env.Bit('windows'):
+ # Copy SwiftShader to plugin dir.
+ swiftshader_path = env.subst('$SWIFTSHADER_DIR/swiftshader_d3d9.dll')
+ if os.path.exists(swiftshader_path):
+ swiftshader_install = env.Replicate('$ARTIFACTS_DIR/O3DExtras',
+ swiftshader_path)
# -------------------------------------------------------------------------
# Unit tests
diff --git a/o3d/tests/common/win/testing_common.cc b/o3d/tests/common/win/testing_common.cc
index e51fc6a..702f970 100644
--- a/o3d/tests/common/win/testing_common.cc
+++ b/o3d/tests/common/win/testing_common.cc
@@ -160,7 +160,8 @@ int WINAPI WinMain(HINSTANCE instance,
display_window = display_window;
display_window->set_hwnd(g_window_handle);
g_display_window = display_window;
- bool offscreen = ::GetEnvironmentVariableW(kOffScreenRenderer, NULL, 0);
+ bool offscreen = (::GetEnvironmentVariableW(kOffScreenRenderer,
+ NULL, 0) == 0);
if (offscreen) {
success =
g_renderer->Init(*g_display_window, true) == o3d::Renderer::SUCCESS;
diff --git a/o3d/utils/utils.gyp b/o3d/utils/utils.gyp
index a193113..9fb5e45 100644
--- a/o3d/utils/utils.gyp
+++ b/o3d/utils/utils.gyp
@@ -44,15 +44,20 @@
},
{
'target_name': 'o3dUtilsTest',
- 'type': 'static_library',
- 'sources': [
- "cross/file_path_utils_test.cc",
- "cross/file_text_reader_test.cc",
- "cross/json_writer_test.cc",
- "cross/string_reader_test.cc",
- "cross/string_writer_test.cc",
- "cross/temporary_file_test.cc",
+ 'type': 'none',
+ 'dependencies': [
+ 'o3dUtils',
],
+ 'direct_dependent_settings': {
+ 'sources': [
+ "cross/file_path_utils_test.cc",
+ "cross/file_text_reader_test.cc",
+ "cross/json_writer_test.cc",
+ "cross/string_reader_test.cc",
+ "cross/string_writer_test.cc",
+ "cross/temporary_file_test.cc",
+ ],
+ },
},
],
}