SWGEmu SWGEMU-272

Snoop + datapad access

Under Review for 11 months (due 11 months ago)

  •  
  •  
  •  
  •  
  • Author
  • Moderator
  • Reviewers

SWGEMU-272 9

Summarize the review outcomes (optional)
 
#permalink

Details

Warning: no files are visible, they have all been filtered.
Participant Role Time Spent Comments Latest Comment
Author 9m 1 I'm currently on vacations and can't really code right no...
Moderator 0m    
Reviewer - 100% complete 3m 2 We have started to use Gerrit for reviews instead of Fish...
Reviewer - 0% complete      
Reviewer - 0% complete      
Reviewer - 100% complete 3m 5 Always check for NULL.
Reviewer - 100% complete 3m 1 Is this still active? If so, please fix it per crush's co...
oru
Reviewer - 0% complete      
Reviewer - 0% complete      
Reviewer - 0% complete      
Reviewer - 0% complete      
Total   17m 9  
#permalink

Objectives

This way we can see into player's datapad.

#permalink

Issues Raised From Comments

Key Summary State Assignee
#permalink

General Comments

13 Jul 12

cRush says:

What is this patch about?

24 Jul 12

dannuic says:

Is this still active? If so, please fix it per crush's comments

26 Jul 12

KarlDeux says:

I'm currently on vacations and can't really code right now but i'll take a looks as fast as i'm active again.

01 Feb

Loshult says:

Can you please fix the comments that Crush have posted?

17 Mar

Loshult says:

We have started to use Gerrit for reviews instead of Fishey. Please follow the instructions on the following place: https://www.assembla.com/spaces/swgemu/wiki/Git_Gerrit_and_Jenkins and push your patch to Gerrit instead.

/src/server/.../commands/SnoopCommand.h Changed   4
Open in IDE #permalink

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create Issue

X
Assign To Me

Log time against