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

 
 

bug #31010: Error on MOVE(0)

Submitted by:  Dmitry Zenovich <dzenovich>
Submitted on:  Fri 10 Sep 2010 04:36:37 PM UTC  
 
Category: NoneSeverity: 4 - Important
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Jose Antonio Ortega Ruiz <jao>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission 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>
Project AdministratorIn charge of this item.
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:

Hmm? I see it here:

http://git.savannah.gnu.org/cgit/mdk.git/commit/?id=2d5550c873727745c76e9c68320eeb863e076692

(it's just a one line deletion)

Jose Antonio Ortega Ruiz <jao>
Project AdministratorIn charge of this item.
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>
Project AdministratorIn charge of this item.
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>
Project AdministratorIn charge of this item.
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>
Project AdministratorIn charge of this item.
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 File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 10 Sep 2010 08:54:02 PM UTCjaoStatusConfirmed=>Fixed
      Open/ClosedOpen=>Closed
    Fri 10 Sep 2010 06:28:39 PM UTCjaoSeverity3 - Normal=>4 - Important
    Fri 10 Sep 2010 06:26:22 PM UTCjaoStatusNone=>Confirmed
      Assigned toNone=>jao
    Fri 10 Sep 2010 04:36:37 PM UTCdzenovichCarbon-Copy-=>Added dzenovich

    Back to the top


    Powered by Savane 3.1-cleanup1