From: SourceForge.net <no...@so...> - 2007-01-15 18:31:40
|
Bugs item #1634405, was opened at 2007-01-12 14:17 Message generated for change (Comment added) made by nobody You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=112867&aid=1634405&group_id=12867 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: leouser (leouserz) Assigned to: Nobody/Anonymous (nobody) Summary: Patches for BUG: 1605847 Initial Comment: See: http://sourceforge.net/tracker/index.php?func=detail&aid=1605847&group_id=12867&atid=112867 Ok here comes some patches. ---------------------------------------------------------------------- Comment By: Nobody/Anonymous (nobody) Date: 2007-01-15 10:31 Message: Logged In: NO I assume this should have been submitted into http://jython.org/patches/. Same is true also for http://jython.org/bugs/1636030. But obviously a patch in a bit wrong place is much better than no patch. =) ---------------------------------------------------------------------- Comment By: leouser (leouserz) Date: 2007-01-12 14:50 Message: Logged In: YES user_id=1277399 Originator: YES File Added: BytecodeLoaderDiff2.txt ---------------------------------------------------------------------- Comment By: leouser (leouserz) Date: 2007-01-12 14:49 Message: Logged In: YES user_id=1277399 Originator: YES yes, moving the code into the constructor works. Attached is the Module diff and following will be the ByteCodeloader diff 2. File Added: ModuleDiff2.txt ---------------------------------------------------------------------- Comment By: leouser (leouserz) Date: 2007-01-12 14:29 Message: Logged In: YES user_id=1277399 Originator: YES another possibility is to move all the initialise code into the constructor for the module. It would reduce the code down to one method and make the BytecodeLoader patch simpler. ---------------------------------------------------------------------- Comment By: leouser (leouserz) Date: 2007-01-12 14:21 Message: Logged In: YES user_id=1277399 Originator: YES org.python.compiler.PyFunctionTable patch should be org.python.code.PyFunctionTable patch ok the patch changes the static initialiser to a instance method. This allows the system to provide a String file name to the module. ---------------------------------------------------------------------- Comment By: leouser (leouserz) Date: 2007-01-12 14:20 Message: Logged In: YES user_id=1277399 Originator: YES File Added: impDiff.txt ---------------------------------------------------------------------- Comment By: leouser (leouserz) Date: 2007-01-12 14:19 Message: Logged In: YES user_id=1277399 Originator: YES File Added: PyFunctionTableDiff.txt ---------------------------------------------------------------------- Comment By: leouser (leouserz) Date: 2007-01-12 14:19 Message: Logged In: YES user_id=1277399 Originator: YES File Added: PyDiff.txt ---------------------------------------------------------------------- Comment By: leouser (leouserz) Date: 2007-01-12 14:18 Message: Logged In: YES user_id=1277399 Originator: YES File Added: BytecodeLoaderDiff.txt ---------------------------------------------------------------------- Comment By: leouser (leouserz) Date: 2007-01-12 14:17 Message: Logged In: YES user_id=1277399 Originator: YES File Added: jythonDiff.txt ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=112867&aid=1634405&group_id=12867 |