I thought she did a great job at putting things together, but my only complaint is the deaths.
I saw no point in killing poor Hedwig, who was in a cage and couldn't even fly away. I also cried when Dobby got killed. He really loved Harry, and you really could tell Harry cared for him. And why oh why did she have to kill BOTH Remus and Tonks! They just had a baby!
But they were at war, and people die, and I suppose she was going for a lesson there.
I was very happy that Snape was at last vindicated, and it really touched me when he asked Harry to look at him, wanting Harry's eyes (which are just like his mother's) to be the last thing he saw before he died.