bugBattle for Wesnoth - Bugs: bug #8961, Showing a diff instead of the...

 
 
Show feedback again

bug #8961: Showing a diff instead of the complete config when a data missmatch occurs

Submitted by:  Nils Kneuper <ivanovic>
Submitted on:  Tue 17 Apr 2007 08:29:56 PM UTC  
 
Category: Feature RequestSeverity: 3 - Normal
Priority: 3 - LowItem Group: Networking
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open
Release: 1.3.1+svn-r16874Operating System: Any

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sun 22 Jul 2007 04:33:36 PM UTC, comment #2:

The recruit info is now send to the engine logger at the info level. This means it's slightly easier to get the wanted info, but still requires both players to enable the logging.

Mark de Wever <mordante>
Project Member
Fri 20 Apr 2007 09:48:07 PM UTC, comment #1:

That is not possible with the current setup. The way it works is 1) You get the checksum from the network. 2) You compare it to your own checksum. 3) They differ. 4) Your program prints out the unit cfg so you can compare it with the remote unit cfg later (assuming the person with the remote cfg is someone willing to assist the bug investigation).

However, I agree there should be some way to invoke a more detailed comparison and print the difference, since that would be very valuable tool for debugging OOS.

Patrick P <sapient>
Project Member
Tue 17 Apr 2007 08:29:56 PM UTC, original submission:

When there is a data-missmatch in an mp game when recruiting (IIRC the only time checksums for the units are exchanged) the whole unit config is printed to stderr. I don't know if the own config (does not make much sense since you already have it on your hd) is shown or the one of the oposite side.
If the own data is shown it does not make any sense and should not be printed out. If the one from the oposite is shown, I would prefer just getting a diff to my own data. This way it would be easier to see what exactly is wrong and there would be a little less "textflood" in the output (cf #8941 to get an example of such a flood).

Nils Kneuper <ivanovic>
Project Administrator

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by mordante (Posted a comment)
  • -unavailable- added by sapient (Posted a comment)
  • -unavailable- added by ivanovic (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    No Changes Have Been Made to This Item
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup