duplicate zruty tile

View: New views
2 Messages — Rating Filter:   Alert me  

duplicate zruty tile

by Clive Crous :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi all,

tile 225 "monster, yeti\, ape or other large beast, zruty"
tile 400 "monster, yeti\, ape or other large beast, zruty"

What are the implications of "fixing" this ?

I mean besides the obvious 'tileset will change' issue.
Is there any reason this duplication was left in there ?

imho 225 should be removed.

Thanks

--
Clive Crous
DarkArts Studios
http://www.darkarts.co.za/


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Slashem-devel mailing list
Slashem-devel@...
https://lists.sourceforge.net/lists/listinfo/slashem-devel

attachment0 (194 bytes) Download Attachment

Re: duplicate zruty tile

by J. Ali Harlow-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On 2006-07-19 08:02:35 AM, Clive Crous wrote:

> Hi all,
>
> tile 225 "monster, yeti\, ape or other large beast, zruty"
> tile 400 "monster, yeti\, ape or other large beast, zruty"
>
> What are the implications of "fixing" this ?
>
> I mean besides the obvious 'tileset will change' issue.
> Is there any reason this duplication was left in there ?
>
> imho 225 should be removed.

It's there because the old entry for zruty in monst.c is commented out  
rather than removed (you'll also find an entry in monsters.txt and  
tilemap.c for the same reason). I see no reason why we couldn't just  
remove all of these references in HEAD. Something may break, but that  
just means we have to fix it. I wouldn't want to change 0.0.7 at this  
point.

Hope this helps,

Ali.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Slashem-devel mailing list
Slashem-devel@...
https://lists.sourceforge.net/lists/listinfo/slashem-devel