fix formatting/function naming, and set submodules as attrs on parent modules.

also, add a test that includes importing both a module with __init__.py and a submodule.

src/runtime/import.cpp

@@ -53,7 +53,7 @@ static BoxedModule* createAndRunModule(const std::string& name, const std::strin
 #define LLVM_SYS_FS_EXISTS_CODE_OKAY(code) (!(code))
 #endif
 
-static bool path_exists (const std::string& path) {
+static bool pathExists(const std::string& path) {
 #if LLVMREV < 217625
     bool exists;
     llvm_error_code code = llvm::sys::fs::exists(path, exists);
@@ -76,7 +76,7 @@ static BoxedModule* importPackageFromDirectory(const std::string& name, const st
     if (VERBOSITY() >= 2)
         printf("Searching for %s at %s...\n", name.c_str(), fn.c_str());
 
-    if (!path_exists(fn))
+    if (!pathExists(fn))
         return NULL;
 
     if (VERBOSITY() >= 1)
@@ -95,7 +95,7 @@ static BoxedModule* importFile(const std::string& name, const std::string& path)
     if (VERBOSITY() >= 2)
         printf("Searching for %s at %s...\n", name.c_str(), fn.c_str());
 
-    if (!path_exists(fn))
+    if (!pathExists(fn))
         return NULL;
 
     if (VERBOSITY() >= 1)
@@ -132,13 +132,15 @@ static Box* importSub(const std::string* name, Box* parent_module) {
         BoxedModule* module;
 
         module = importPackageFromDirectory(*name, dn);
-        if (module)
-            return module;
-
+        if (!module)
             module = importFile(*name, dn);
-        if (module)
+
+        if (module) {
+            if (parent_module)
+                parent_module->setattr(*name, module, NULL);
             return module;
         }
+    }
 
     if (*name == "basic_test") {
         return importTestExtension("basic_test");

test/tests/package_import.py

+import distutils
+import distutils.log
+print type(distutils)
+print type(distutils.log)