bugGNU MIX Development Kit - Bugs: bug #31010, Error on MOVE(0)

 
 

bug #31010: Error on MOVE(0)

Submitter:  Dmitry Zenovich <dzenovich>
Submitted:  Fri 10 Sep 2010 04:36:37 PM UTC
   
 
Category:  None Severity:  4 - Important
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  jao
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Sat 11 Sep 2010 07:24:41 PM UTC, comment #12: 

Thanks!

Dmitry Zenovich <dzenovich>
Sat 11 Sep 2010 07:00:50 PM UTC, comment #11: 

Thanks a lot for testing, Dmitry. I've set the minimum libintl to 0.37 (i just needed something better than 0.30).

Jose Antonio Ortega Ruiz <jao>
Group administrator
Sat 11 Sep 2010 06:49:25 PM UTC, comment #10: 

Oh, I'd forgotten to do 'git pull'. Now the mixvm is alright. Thank you very much!

PS. Could you please edit the the configure.ac file as in this diff:
index a4397d3..179297d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -42,7 +42,7 @@ fi
 dnl i18n stuff
 AM_GNU_GETTEXT_VERSION(0.14)
 AM_GNU_GETTEXT
-AC_PROG_INTLTOOL([0.40], [no-xml])
+AC_PROG_INTLTOOL([0.37], [no-xml])

 GETTEXT_PACKAGE=mdk
 AC_SUBST(GETTEXT_PACKAGE)

Is there any sense for using intltool 0.40 (my Ubuntu has only 0.37)?

Dmitry Zenovich <dzenovich>
Sat 11 Sep 2010 06:30:07 PM UTC, comment #9: 
Jose Antonio Ortega Ruiz <jao>
Group administrator
Sat 11 Sep 2010 06:25:37 PM UTC, comment #8: 

I can't see your commit :(

Dmitry Zenovich <dzenovich>
Sat 11 Sep 2010 05:37:20 PM UTC, comment #7: 

Ouch, sorry Dmitry! Can you please try again and see if it works now?
Thanks!

Jose Antonio Ortega Ruiz <jao>
Group administrator
Sat 11 Sep 2010 04:50:55 PM UTC, comment #6: 

Sorry for repeats.

Dmitry Zenovich <dzenovich>
Sat 11 Sep 2010 04:19:22 PM UTC, comment #5: 

Thank you. Now the MOVE(0) seems to work correctly, but the mixvm doesn't anymore step to the next line after doing non-zero MOVEs:

Program loaded. Start address: 1000
MIX> pline
Line 2: START       STZ     0
MIX> next
Elapsed time: 2 /Total program time: 2 (Total uptime: 2)
MIX> pline
Line 3:             ENT1    1
MIX> next
Elapsed time: 1 /Total program time: 3 (Total uptime: 3)
MIX> pline
Line 4:             MOVE    0(0)
MIX> next
Elapsed time: 1 /Total program time: 4 (Total uptime: 4)
MIX> pline
Line 5:             MOVE    0(36)
MIX> next
Elapsed time: 73 /Total program time: 77 (Total uptime: 77)
MIX> pline
Line 5:             MOVE    0(36)
MIX> next
Elapsed time: 73 /Total program time: 150 (Total uptime: 150)
MIX> pline
Line 5:             MOVE    0(36)
MIX> next
Elapsed time: 73 /Total program time: 223 (Total uptime: 223)
MIX> pline next
Line 5:             MOVE    0(36)
MIX> run
Running ...
... done
Elapsed time: 4822 /Total program time: 5045 (Total uptime: 5045)

Dmitry Zenovich <dzenovich>
Sat 11 Sep 2010 04:17:20 PM UTC, comment #4: 

Thank you. Now the MOVE(0) seems to work correctly, but the mixvm doesn't anymore step to the next line after doing non-zero MOVEs:

Program loaded. Start address: 1000
MIX> pline
Line 2: START       STZ     0
MIX> next
Elapsed time: 2 /Total program time: 2 (Total uptime: 2)
MIX> pline
Line 3:             ENT1    1
MIX> next
Elapsed time: 1 /Total program time: 3 (Total uptime: 3)
MIX> pline
Line 4:             MOVE    0(0)
MIX> next
Elapsed time: 1 /Total program time: 4 (Total uptime: 4)
MIX> pline
Line 5:             MOVE    0(36)
MIX> next
Elapsed time: 73 /Total program time: 77 (Total uptime: 77)
MIX> pline
Line 5:             MOVE    0(36)
MIX> next
Elapsed time: 73 /Total program time: 150 (Total uptime: 150)
MIX> pline
Line 5:             MOVE    0(36)
MIX> next
Elapsed time: 73 /Total program time: 223 (Total uptime: 223)
MIX> pline next
Line 5:             MOVE    0(36)
MIX> run
Running ...
... done
Elapsed time: 4822 /Total program time: 5045 (Total uptime: 5045)

Dmitry Zenovich <dzenovich>
Fri 10 Sep 2010 08:54:02 PM UTC, comment #3: 

I've just pushed a patch [0] to master that should fix this problem. (My previous comment was silly, since the vm does not emit warnings---i thought at first it was the assembler complaining).

Please, reopen this bug if it doesn't work for you!

[0] commit 7e5ea91374a8481956082e9d1b98dc9f35931822

Jose Antonio Ortega Ruiz <jao>
Group administrator
Fri 10 Sep 2010 08:37:04 PM UTC, comment #2: 

Yes, I think it is acceptable, although the book points that instruction should not do anything in that case, and there is no mentions about warnings.

Dmitry Zenovich <dzenovich>
Fri 10 Sep 2010 06:28:39 PM UTC, comment #1: 

I think that i will convert the error to a warning. Dmitry (or anyone else), would you find that acceptable?

Jose Antonio Ortega Ruiz <jao>
Group administrator
Fri 10 Sep 2010 04:36:37 PM UTC, original submission:  

According to "The Art of Computer Programming", the MIX instruction MOVE should not do anything when F=0, no error should happen. Nevertheless the mixvm produces the error message and stops running a program:

ERROR: 1002: [MOVE      0,0(0:0)  ]                 MOVE    0(0)

Dmitry Zenovich <dzenovich>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by jao (Updated the item)
  • -email is unavailable- added by dzenovich (Submitted the item)
  • -email is unavailable- added by dzenovich
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2010-09-10 jao StatusConfirmed Fixed
        Open/ClosedOpen Closed
    2010-09-10 jao Severity3 - Normal 4 - Important
    2010-09-10 jao StatusNone Confirmed
        Assigned toNone jao
    2010-09-10 dzenovich Carbon-Copy- Added dzenovich

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code