SUBSIM Radio Room Forums



SUBSIM: The Web's #1 resource for all submarine & naval simulations since 1997

Go Back   SUBSIM Radio Room Forums > Silent Hunter 3 - 4 - 5 > SHIII Mods Workshop
Forget password? Reset here

Reply
 
Thread Tools Display Modes
Old 06-02-19, 09:58 AM   #1
Fader_Berg
中国水兵
 
Join Date: Jun 2009
Location: Uppland, Sweden
Posts: 278
Downloads: 48
Uploads: 0
Default

Quote:
Originally Posted by Blackswan1940 View Post
Ahaaa! And you should also consider taking back your last decision about rekoving the compatibilty to H.sie!

It is totally counterproductive when your patch is not compatible with Hasie and Stiebler. It will not be used then.

Hope this thought helps. Keep up your good work!
No, I should not. Look. Hsies code is horrible to work with. It's all over the place and he has made many poor design choices. It's too *******ing boring and time consuming to work with. Besides... I don't use many of his patches anyway.

Counterproductive...? I take that as an insult. Do you even know what that word mean? I may possibly have made something that you don't want to use. I use it though. That's pretty much all that matters for me. If any one else wants to use it - fine. If anyone else don't want to use it - perfect. I don't care.

I have given all the code away into the public domain for anyone to fiddle with and to merge with hsies work if that's wanted. How is that counterproductive?! Blame yourself or anyone else that have done nothing in this matter for being counterproductive instead.

D*mn I'm pissed... counterproductive my *ss.
__________________
patSH3r-developer, (https://fb.tuxxor.net)
Type II junkie

Last edited by Fader_Berg; 06-02-19 at 10:08 AM.
Fader_Berg is offline   Reply With Quote
Old 06-02-19, 10:15 AM   #2
padi
Officer
 
Join Date: Sep 2012
Posts: 249
Downloads: 126
Uploads: 0
Default

Quote:
Originally Posted by Fader_Berg View Post
No, I should not. Look. Hsies code is horrible to work with. It's all over the place and he has made many poor design choices. It's too *******ing boring and time consuming to work with. Besides... I don't use many of his patches anyway.

Counterproductive...? I take that as an insult. Do you even know what that word mean? I may possibly have made something that you don't want to use. I use it though. That's pretty much all that matters for me. If any one else wants to use it - fine. If anyone else don't want to use it - perfect. I don't care.

I have given all the code away into the public domain for any one to fiddle with and to merge with hsies work if that's wanted. How is that counterproductive?! Blame yourself or anyone else that have done nothing in this matter for being counterproductive instead.

D*mn I'm pissed... counterproductive my *ss.

Creating something that isn't compatible to the standard is counterproductive because very few people are using it...
__________________
My Mediafire Page

Join the Real ASW Mod Discord-Server:
https://discord.gg/Yg5ZE25
padi is offline   Reply With Quote
Old 06-02-19, 10:25 AM   #3
Fader_Berg
中国水兵
 
Join Date: Jun 2009
Location: Uppland, Sweden
Posts: 278
Downloads: 48
Uploads: 0
Default

Quote:
Originally Posted by padi View Post
Creating something that isn't compatible to the standard is counterproductive because very few people are using it...
The fact that it's created for a small audience (if any at all) doesn't make it counterproductive. I'm sorry.
__________________
patSH3r-developer, (https://fb.tuxxor.net)
Type II junkie
Fader_Berg is offline   Reply With Quote
Old 06-02-19, 03:24 PM   #4
propbeanie
CTD - it's not just a job
 
propbeanie's Avatar
 
Join Date: May 2016
Location: One hour from Music City USA!
Posts: 9,762
Downloads: 441
Uploads: 2


Default

I have a quote in my signature from Jimbuna that perfectly fits this. I would also like to thank Fader_Berg for what he has accomplished with this. I would also like to thank Stiebler and h.sie for what they have done - in fact, I'd like to thank ~ALL~ the modders who have done anything for the modding community, conforming to "standards" or not (by whose definition are these "standards", btw?). It ain't easy doing this modding stuff, because as can be seen, someone seems quick to criticize. Do it constructively and politely please... whoops - my soapbox collapsed from my body weight... sorry.
__________________

"...and bollocks to the naysayer/s" - Jimbuna
propbeanie is offline   Reply With Quote
Old 06-03-19, 03:00 PM   #5
areo16
Sonar Guy
 
Join Date: Oct 2013
Posts: 377
Downloads: 64
Uploads: 0
Default

Quote:
Originally Posted by Fader_Berg View Post
No, I should not. Look. Hsies code is horrible to work with. It's all over the place and he has made many poor design choices. It's too *******ing boring and time consuming to work with. Besides... I don't use many of his patches anyway.

Counterproductive...? I take that as an insult. Do you even know what that word mean? I may possibly have made something that you don't want to use. I use it though. That's pretty much all that matters for me. If any one else wants to use it - fine. If anyone else don't want to use it - perfect. I don't care.

I have given all the code away into the public domain for anyone to fiddle with and to merge with hsies work if that's wanted. How is that counterproductive?! Blame yourself or anyone else that have done nothing in this matter for being counterproductive instead.

D*mn I'm pissed... counterproductive my *ss.

I do sympathize here as concerns to working with other people's code that may not have the same disciplines as yourself, same education or even understanding of design principles.



It's quite frustrating (not to say that either of those developers in question used bad practices as I haven't spent much time looking at the source. I do know however, that working with assembly is so low-level that any design concepts, if they can be applied, that are common with higher level languages can't be applied to assembly.). I'd really hate to design a whole application in assembly to begin with, why not read the byte values in a higher level language and output the modified bytes to executable?
areo16 is offline   Reply With Quote
Old 06-03-19, 04:31 PM   #6
Fader_Berg
中国水兵
 
Join Date: Jun 2009
Location: Uppland, Sweden
Posts: 278
Downloads: 48
Uploads: 0
Default

Quote:
Originally Posted by areo16 View Post
I do sympathize here as concerns to working with other people's code that may not have the same disciplines as yourself, same education or even understanding of design principles.
hsie made a great deal of work that impressed me and others. I really don't care how his code looks like while it works and I don't need to get in touch with it.


Quote:
Originally Posted by areo16 View Post
It's quite frustrating (not to say that either of those developers in question used bad practices as I haven't spent much time looking at the source. I do know however, that working with assembly is so low-level that any design concepts, if they can be applied, that are common with higher level languages can't be applied to assembly.). I'd really hate to design a whole application in assembly to begin with, why not read the byte values in a higher level language and output the modified bytes to executable?
The problem is where the two patches interfere with each other. It can't be avoided and has to be dealt with.
I tried at first. In some parts of the patSH3r source code you can still see that I tried it. It's mentioned in the comments (which I've forgotten to update when I gave up).


If I make one patch I might end up having to write two or more patches to make it compatible with hsies. One when hsie.A i enabled. One more when hsie.A is disabled. Fu*k... it crashes. This because a variable in the hsie.A-code are also being used in hsie.C and -F-code too, and doesn't get updated correctly now. I have to figure out how this variable works in fragmented, disorganized and very sparse commented code in gerrman. By the way... which other hsie-features uses this variable also?! Not to mention all variables that are being used between A, B, G and O that I may have missed, and doesn't result in a crash. That's gonna affect the game somehow. It probably ends up in some "when I got 56% oxy left and the clock is 12:42 a friday the submarine crashdives"-bug.
I just puked in my mouth.



No! If I have to fill my my spare time with something extremely hard and boring. I rather work extra and make some money instead.
__________________
patSH3r-developer, (https://fb.tuxxor.net)
Type II junkie

Last edited by Fader_Berg; 06-03-19 at 04:49 PM.
Fader_Berg is offline   Reply With Quote
Old 06-03-19, 05:19 PM   #7
areo16
Sonar Guy
 
Join Date: Oct 2013
Posts: 377
Downloads: 64
Uploads: 0
Default

Quote:
Originally Posted by Fader_Berg View Post
If I make one patch I might end up having to write two or more patches to make it compatible with hsies. One when hsie.A i enabled. One more when hsie.A is disabled. Fu*k... it crashes. This because a variable in the hsie.A-code are also being used in hsie.C and -F-code too, and doesn't get updated correctly now. I have to figure out how this variable works in fragmented, disorganized and very sparse commented code in gerrman. By the way... which other hsie-features uses this variable also?! Not to mention all variables that are being used between A, B, G and O that I may have missed, and doesn't result in a crash. That's gonna affect the game somehow. It probably ends up in some "when I got 56% oxy left and the clock is 12:42 a friday the submarine crashdives"-bug.
I just puked in my mouth.

This sounds like is using global variables here when he should be using local variables? Or he is referencing the same variable from the same package over and over again?


This is a design issue.


What about copying these globals to local variables in the scope you need to use it in, and edit them as such, not affect the global variable itself (assuming you don't need the edits on these variables to be outside of your current scope)?
areo16 is offline   Reply With Quote
Old 06-03-19, 10:17 PM   #8
Fader_Berg
中国水兵
 
Join Date: Jun 2009
Location: Uppland, Sweden
Posts: 278
Downloads: 48
Uploads: 0
Default

Quote:
Originally Posted by areo16 View Post
This sounds like is using global variables here when he should be using local variables? Or he is referencing the same variable from the same package over and over again?


This is a design issue.


What about copying these globals to local variables in the scope you need to use it in, and edit them as such, not affect the global variable itself (assuming you don't need the edits on these variables to be outside of your current scope)?
Some of them are global for a reason, some of them are not. Believe me. If there was a easy way around this, I would have taken it a long time ago.

To be able to remedy this I (or anyone else for that matter) need to analyze and understand pretty much everything he has written. Since, like I told you, the code is very sparsely commented in german (with even most of the variable names being made in german, scattered around the source in a non logical way). it's damn frustrating and hard to cope with.

It's not impossible off course, Anyone can do it. It just takes time and effort I'm not willing to invest.
__________________
patSH3r-developer, (https://fb.tuxxor.net)
Type II junkie

Last edited by Fader_Berg; 06-03-19 at 10:49 PM.
Fader_Berg is offline   Reply With Quote
Reply


Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT -5. The time now is 10:03 AM.


Powered by vBulletin® Version 3.8.11
Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.
Copyright © 1995- 2024 Subsim®
"Subsim" is a registered trademark, all rights reserved.