Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/49420 )

Change subject: python: Remove python/embedded.cc's dependence on the rest of gem5.
......................................................................

python: Remove python/embedded.cc's dependence on the rest of gem5.

If embedded python modules are going to be built into other binaries,
then they can't depend on other parts of gem5.

Change-Id: I4e12f1962cb6330e70866a40388689102c23693a
---
M src/python/embedded.cc
1 file changed, 2 insertions(+), 4 deletions(-)



diff --git a/src/python/embedded.cc b/src/python/embedded.cc
index 27bf770..21d6a0d 100644
--- a/src/python/embedded.cc
+++ b/src/python/embedded.cc
@@ -45,10 +45,9 @@

 #include <zlib.h>

+#include <cassert>
 #include <list>

-#include "base/logging.hh"
-
 namespace py = pybind11;

 namespace gem5
@@ -78,8 +77,7 @@
     Bytef marshalled[len];
     uLongf unzlen = len;
     int ret = uncompress(marshalled, &unzlen, (const Bytef *)code, zlen);
-    if (ret != Z_OK)
-        panic("Could not uncompress code: %s\n", zError(ret));
+    assert(ret == Z_OK);
     assert(unzlen == (uLongf)len);

     auto marshal = py::module_::import("marshal");

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/49420
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I4e12f1962cb6330e70866a40388689102c23693a
Gerrit-Change-Number: 49420
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to