Author Topic: Money, cash greenback  (Read 14047 times)

0 Members and 4 Guests are viewing this topic.

Offline robbo007

Re: Money, cash greenback
« Reply #15 on: August 20, 2025, 12:41:03 PM »
SCI Companion shows the group number in hex for some reason. In that case, you need to find group numbers 58 (hex) and 59 (hex) instead.

OMG!!!  You can go over code 100 times and not see something thats in plain daylight.

Hex code 58,59 in the Vocab file pointed me to the area and I saw this. The closing parenthesis on my inventory item should not be there. Removing it resolved the Bad said spec.

Code: [Select]
(instance CROWEBAR of Iitem
(properties
said '/crowebar, crowbar)'
description 101 ;The Crowebar \05_ This thing looks tough.
owner 0
view 227
loop 0
cel 0
script 0
name "Crowebar \05_"
)
)

Thanks for all your suggestions everybody and sorry for the long post...

Offline Kawa

Re: Money, cash greenback
« Reply #16 on: August 20, 2025, 12:50:41 PM »
OMG!!!  You can go over code 100 times and not see something thats in plain daylight.
C++ game project, tried to load a 3D model with more than 50 bones in a structure that only has space for that many bones. Drove me damn near insane for a week before someone found out on my behalf that that was the issue.

Offline lskovlun

Re: Money, cash greenback
« Reply #17 on: August 20, 2025, 01:11:05 PM »
Kawa, maybe we need to make this print group numbers in decimal? The use cases for hex notation here are rather narrow (mainly related to the special tokens which are located at the top of the 12-bit range). And the interpreter uses decimal.

Offline lskovlun

Re: Money, cash greenback
« Reply #18 on: August 20, 2025, 01:22:35 PM »
BTW, if you don't intend to distinguish crowebar and crowbar anywhere, you can just give them a single group number. That's kind of the point of them.

Offline Kawa

Re: Money, cash greenback
« Reply #19 on: August 20, 2025, 01:47:23 PM »
Kawa, maybe we need to make this print group numbers in decimal? The use cases for hex notation here are rather narrow (mainly related to the special tokens which are located at the top of the 12-bit range). And the interpreter uses decimal.
Ten-four, supreme space cadet sir!

*loads up MSVC*

Update: the next Nightly Build should have this change in it.
« Last Edit: August 20, 2025, 01:54:43 PM by Kawa »

Offline robbo007

Re: Money, cash greenback
« Reply #20 on: August 20, 2025, 02:36:00 PM »
Kawa, maybe we need to make this print group numbers in decimal? The use cases for hex notation here are rather narrow (mainly related to the special tokens which are located at the top of the 12-bit range). And the interpreter uses decimal.
Ten-four, supreme space cadet sir!

*loads up MSVC*

Update: the next Nightly Build should have this change in it.

Any chance your fork of SC can support Windows XP :)

Offline Kawa

Re: Money, cash greenback
« Reply #21 on: August 20, 2025, 02:58:14 PM »
Any chance your fork of SC can support Windows XP :)
I guess there is a setting to make it target XP... and I mean in Visual Studio, not this.

Edit: https://helmet.kafuka.org/sci/SCICompanion_XP.exe freshly made for you. I don't know if it'll work, but lemme know. (Also includes the decimal word groups)
« Last Edit: August 20, 2025, 03:27:20 PM by Kawa »

Offline robbo007

Re: Money, cash greenback
« Reply #22 on: August 20, 2025, 03:42:59 PM »
Any chance your fork of SC can support Windows XP :)
I guess there is a setting to make it target XP... and I mean in Visual Studio, not this.

Edit: https://helmet.kafuka.org/sci/SCICompanion_XP.exe freshly made for you. I don't know if it'll work, but lemme know. (Also includes the decimal word groups)

Amazing thanks. it seems to load and run things fine :)
Wow...I see it detects inconsistencies in the code that he original version doesn't. I'm going to have to check the readme on your github to see what has changed.



Offline Doan Sephim

Re: Money, cash greenback
« Reply #23 on: August 21, 2025, 10:24:06 AM »
Before I forget to mention, there's a parser command in your main script that I suspect doesn't work as you intend.
Code: [Select]
(cond
((Said 'look>')
(cond
((Said '/carpet,down')
"Look carpet" will work, although I'm not sure why this would be in your main script...unless there's a carpet in every room in the game.
But "look down" doesn't parse as "down" I believe is an adverb. I think the game will just give the standard "look" command if the player types "look down."

Just a small observation.

Offline robbo007

Re: Money, cash greenback
« Reply #24 on: August 21, 2025, 10:32:11 AM »
Before I forget to mention, there's a parser command in your main script that I suspect doesn't work as you intend.
Code: [Select]
(cond
((Said 'look>')
(cond
((Said '/carpet,down')
"Look carpet" will work, although I'm not sure why this would be in your main script...unless there's a carpet in every room in the game.
But "look down" doesn't parse as "down" I believe is an adverb. I think the game will just give the standard "look" command if the player types "look down."

Just a small observation.

ahh ok thanks for spotting that :) I'll review it.. It's a luxury to have a second set of eyes on your code :)


SMF 2.0.19 | SMF © 2021, Simple Machines
Simple Audio Video Embedder

Page created in 0.027 seconds with 18 queries.