Mirage Source 4

[Robin]

http://web.miragesource.com/forums/viewt...11&start=0

[Acruno]

Trying to sell Chaos Engine?
I lol'd. Really.

[Robin]

Feel free to go code that.

[Clu]

Hey Dave, uhm, im trying to implement your little scrolling map transition into my source of MSE3.4 whatever xD, and im not exactly sure how to use your patch thing, i got the code by opening it as a txt, but im smart enough to realise thats not the intended purpose, is there any chat i could add you on for help with it, or could you pm me, or just post or link me or something, it would be a huge help, whenever you get a chance i mean, thanks

[Acruno]

It's easy enough to do as a plaintext file if you don't understand Subversion.

Any line with a - before it should be removed, and if it has a + then it should be added.

[Clu]

I realised that lol, but honestly it's an amazingly assload giant pain in the ass to remove all those plusses and minuses while doing it, which i really have no problem with if i have to, i just have no clue what subversion thing is, so i figured if there's an easier way, why waste such time doing a meaningless task, when i can do other things instead *shrugs*, so any type of explanation to this subversion thing would be extremely helpfull, thanks

[Acruno]

Ctrl + H. Use it.

Alternatively, Google. Use it. There are loads of very helpful subversion tutorials on the internet that will help you much more.

[Clu]

Touche, I didnt realise i wasx so out of it i didnt think of Google T.T, thanks i can find out what i need to know from that, thanks for the help

[Anthony]

I see what your saying Dave. I wasn't aware that the decrease of packet size would be so insigificant. I always thought that converting the packet names to an enum would order them into just numbers, or longs.

I will take a look at Verrigans code, I had once before but the entire conversion.. It's a whole lot of work :3. It might be something I would be interested in doing if I get some time but right now I am pretty busy with work.

[Jacob]

If you're gonna use Verrigans system you'll have to fix an issue with IncomingData.

Code:

'Check to make sure we have at least 2 bytes (Integer), so we can
  'check the packet length.
  If aLen(Player(Index).Buffer) >= 2 Then
    pl = GetIntegerFromBuffer(Player(Index).Buffer)
    Player(Index).Buffer = RemoveFromBuffer(Player(Index).Buffer, 2)
  End If

[code] 'While we have our entire packet in the buffer, handle it.
While pl > 0 And pl

[Jacob]

Don't quote me, but I think TCP just means there is no packet loss, doesn't mean it comes all at once.

[Anthony]

I would say maybe reverting back to 3.49 (even though I did spend quite a bit of time on it x.x) would be best for now. Maybe?

Besides that everything that has been done so far is excellent DFA, Mirage has taken a massive jump in organization and optimizations since you started working on it. Is great!

[Acruno]

I noticed that with the timer walking system, every now and then the character sort of freezes for about half a second, then carries on. It seems to be an irregular occurance, but I haven't really looked into it extensively.

[Matt]

It's prolly just you, cause I've never seen that.

[Anthony]

Small bug in Sub CastSpell when trying to cast without a target gives a subscript out of range.

I just added in:

Code:

If TempPlayer(Index).Target = TARGET_TYPE_NONE Then
        Exit Sub
    End If

At the top of the sub.

[Nean]

Errr... Sorry for necro posting, but I just tested this out, and it runs very very well. I think this is what I'm going to use, to start actually delving into VB6.

[Rian]

There's been like 4 other posts in this thread today. :?

[Acruno]

The movement goes, step step freeze step step freeze etc.

[El_Dindonnier]

How to add this :
viewtopic.php?f=75&t=1090 (extra layer)
and
viewtopic.php?f=75&t=797 (Adding Width to the MapEditor)

to MS4 please ? :|

PS: Thanks you DFA for MS4

[Kousaten]

The extra layers tutorial won't work without changes on MS4 because of the optimized surfaces. :mrgreen: You've gotta figure out how the code works before screwing around with it like that. (Or just copy all the ground stuff like Advocate said in a post before.)

As for map editor width, change the .width in it and handle how the client reads it all. The magic number is 7, I believe.

[Kousaten]

Unless you're just making it for a few friends who don't really care about the details.

[GIAKEN]

The first thing I see is you're not using the benefit of putting a sub in it's on module (HandleData). Every time the sub is called it creates all of these variables:

Code:

Dim Parse() As String
Dim Name As String
Dim Password As String
Dim Sex As Long
Dim Class As Long
Dim CharNum As Long
Dim Msg As String
Dim MsgTo As Long
Dim Dir As Long
Dim InvNum As Long
Dim Ammount As Long
Dim Damage As Long
Dim PointType As Long
Dim Movement As Long
Dim i As Long, n As Long, x As Long, y As Long, f As Long
Dim MapNum As Long
Dim s As String
Dim tMapStart As Long, tMapEnd As Long
Dim ShopNum As Long, ItemNum As Long
Dim DurNeeded As Long, GoldNeeded As Long

There really isn't a point to them not being static...so it would be good to just do:

Code:

Private Parse() As String
Private Name As String
Private Password As String
Private Sex As Long
Private Class As Long
Private CharNum As Long
Private Msg As String
Private MsgTo As Long
Private Dir As Long
Private InvNum As Long
Private Ammount As Long
Private Damage As Long
Private PointType As Long
Private Movement As Long
Private i As Long
Private n As Long
Private x As Long
Private y As Long
Private f As Long
Private MapNum As Long
Private s As String
Private tMapStart As Long
Private tMapEnd As Long
Private ShopNum As Long
Private ItemNum As Long
Private DurNeeded As Long
Private GoldNeeded As Long

Also any other subs / functions that get called a lot (like in a loop or whatever) should get static variables.

[Jacob]

http://www.deadnoggin.com/deadnoggin/ms4.rar

+++++++++++++
++ MS 3.51 ++
+++++++++++++
*This release was done by Dugor*

--------
|Client|
--------
- Changed Val() to the appropriate Conversion
- HandleKeyPresses - Added a check for "/" before checking commands
- HandleKeyPresses - Changed how commands are checked with a string array http://web.miragesource.com/forums/viewt...331#p52200
- Finished Equipment Enum
- Fixed leaving map while in mapeditor - Dfa and Nean

--------
|Server|
--------
- Finished Equipment Enum

I don't have enough time to do much more.

[GIAKEN]

I just compared the time it takes with the < 1 check and the = 0 check and they're about the same...

[GIAKEN]

Booleans are a lot faster than any other variable from my testing, too.