[Announce] NPPAngband 0.5.1 WIP 2

[nppangband]

NPP 0.5.1 WIP2 has been uploaded. This is mostly a major clode cleanup. It now compiles cleanly with cygwin with no warnings ( I work with main-win.c.0. Probably about 40-50 bugs were fixed from WIP-1. The main new feature is mouse support for most of the options screens (accessed by '=') using the modern Angband code. This also includes addding the itemlist window from Angband.

People trying to compile using other main-xxx.c files and makefiles should have a much easier time now. Please let me know what changes I need to make for any other makefile or main-xxx.c. And thanks to those who gave WIP-1 a try.

The links are:

source:
http://download.nppangband.org/npp-0.5.1-src-WIP2.zip

Windows executable:
http://download.nppangband.org/npp-0.5.1-win-WIP2.zip



Thanks again to all willing to give this version a test. It should be fairly playable by now, but please save often.

-Jeff

[d_m]

So, this might just be user error, but I wasn't able to compile WIP-2 on Linux.

There were about 6 fatal warnings (mostly redefinitions between externs.h and other files) which I did fix. However, there were *tons* of warnings about unreachable code. Some of these warnings were coming from system libraries like /usr/lib/bits/unistd.h, so I think at least some are bogus.

The compile ultimately failed because it was trying to pull cairo in unconditionally. I think I can get it working but so far no luck.

[d_m]

OK, I got it compiling and running on Linux... I will make a patch for you once it seems to be working OK.

One bug I hit really quickly is that NPP wants to store prefs in $HOME/.angband/NPP0.5.1-WIP2 but doesn't actually create this directory. So after character creation it crashes.

I found that path via strace and I assume that if I manually create the directory it'll be fine, but obviously it would be good for the game to do it automatically.

[nppangband]

Which makefile are you using? It must involve #defines that aren't #defined for main-win.x.

One thing I did between WIP1 and WIP2 is make sure most of the NPP.h files are practically identical to the Angband h files, and the different code moved to separate files. The main difference on all of the z-**** files was all of the extended character code and I moved that to its own file. All the main-xxx files and most of the .h files are straight out of Angband 3.1.2v2.

And again, many thanks for helping me out.

[nppangband]

Originally Posted by d_m

One bug I hit really quickly is that NPP wants to store prefs in $HOME/.angband/NPP0.5.1-WIP2 but doesn't actually create this directory. So after character creation it crashes.

Should there be another director made in init2.c? I thought it had all of the directories that Angband has, plus NPP/lib/bones for player ghosts.

I miss Diego, my NPP co-maintainer, for many reasons, but one is that he works in Linux, and I work in windows, so between the two of us we could catch most of the bugs that happen on the various OSs.

[d_m]

Here are the changes I had to make to get it to compile. Some of these changes are quick and dirty so you probably shoudn't just apply the patch. That said, all the changes were necessary for it to build for me.

[d_m]

Originally Posted by nppangband

Should there be another director made in init2.c? I thought it had all of the directories that Angband has, plus NPP/lib/bones for player ghosts.

I miss Diego, my NPP co-maintainer, for many reasons, but one is that he works in Linux, and I work in windows, so between the two of us we could catch most of the bugs that happen on the various OSs.

This turned out to be a problem related to autotools (used by V but not be NPP). The z-file.c program uses symbols like HAS_MKDIR to make the file handling functions work. So when I'm building NPP those variables aren't set (they get set by ./configure in V I thnk). I added -DHAS_MKDIR to Makefile.std and then the directory got created correctly.

Probably I should change the V code so if you aren't on WINDOWS and don't have HAS_MKDIR then it will crash during compilation. The "default" behavior isn't very helpful (basically everything just fails).

Also, two bugs from playtesting so far:

1. Sometimes when @ runs for a bit it doesn't reappear when the run is over... you have to move manually or do some other action and then the @ sign reappears. It doesn't seem to always happen but it was happening enough that it kept throwing me off.

2. the item name/enscript in shops gets drawn under the weight, so if you have an item with a long name/inscription the closing part of the inscription can actually appear to the right of the weigth ("behind" it).

[nppangband]

Originally Posted by d_m

Here are the changes I had to make to get it to compile. Some of these changes are quick and dirty so you probably shoudn't just apply the patch. That said, all the changes were necessary for it to build for me.

Again, thank you. Most of those changes involved redundant delcarations of functions or variables. As for the changes to the makefiles, I assume the long-term solution is to add the src/gtk/ subdirectory where the cairo files are located. I added them.

[nppangband]

Originally Posted by d_m

Also, two bugs from playtesting so far:

1. Sometimes when @ runs for a bit it doesn't reappear when the run is over... you have to move manually or do some other action and then the @ sign reappears. It doesn't seem to always happen but it was happening enough that it kept throwing me off.

I noticed that too. I fixed the problem. If it is really annoying you, the fix is in cave.c in map_info. Around line 1505, this line should read (delete the second condition about running):

/* Handle "player" */
else if (m_idx < 0)

Not that it affects gameplay, but right after that, around line 1560, near the end of map_info, there is a related "else if" statement that can be completely deleted. This bug came up when I removed the "hide player while running" option.

Originally Posted by d_m

2. the item name/enscript in shops gets drawn under the weight, so if you have an item with a long name/inscription the closing part of the inscription can actually appear to the right of the weigth ("behind" it).

Yes, some of the calls to object_desc are wrong. This is because I am still using the old object_desc function from Angband 3.0.6. Sometime between 3.0.6 and 3.1.2 it got a re-write, and where I incorporated updated Angband code I haven't always called the function properly. Sometimes an object name is going to print with either too much or too little information. I am updating object_desc this week, and then these bugs should go away.

[ekolis]

I can't get help on options: "cannot load option.txt" - the file is called options.txt, not option.txt!

edit: also, the "Please choose an item to examine" message in shops behaves weird when you get it several times in a row... and where is the "depth in feet" option hiding?! I wanna turn it off!

And I found a "rapier of slays" in the store... the examine screen says it slays orcs though...

And I tried to buy 99 arrows costing $1 apiece when I had $127... but I can't afford it, according to the game! What is this, fractional gold? When I bought twenty of them, I wound up with 98 gold remaining, so I had somehow spent 29 gold total... WTF?!

Do druids have some hidden combat penalties or something? I hit a black naga with one out of ten of my arrows, and could barely scratch anything with a dagger, but I still had +0,+0 on the char screen!

The set-trap command seems to be set to autorepeat, which is quite annoying!

Sometimes, when running, @ randomly disappears...

I was confused by a scintillating colored gas trap while BLIND? What, is it not just the colors that are confusing?