I haven't read the book since it came out and my memory of it is a bit hazy. It's not necessarily a bad thing as it helps me assess the movie from a fresher perspective. Random knit picks:
The first part of the movie didn't really work for me. I just couldn't get into it. How abysmally did the trio plan the Gringott's heist? Everything from Hermione not knowing how Bellatrix would act despite being tortured by her several hours ago, down to no one knowing anything about the bank's defenses, etc.
There should have been more build up regarding the trio's return to Hogwarts. It seemed terribly easy and all rather ho-hum.
They took time to mention that Death Eaters were on the Hogwarts staff, but they didn't do anything.
The Hogwarts staff was pretty thin. McGonagall, Flitwick, Sprout, and Slughorn. The one shot of Trelawney doesn't count. Hagrid was notably absent from all proceedings. Where the heck was this dude? I imagine there's a deleted scene somewhere of Death Eaters capturing him before the battle, but I'm surprised he played no part in setting up Hogwart's defenses.
On the horcruxes, Harry and Dumbledore figure out that Voldemort wanted to make 7, but the trio arbitrarily decided to stop counting at 5 or 6?
They short-shrifted the Dumbledore storyline. I personally feel they could've cut it all out, but considering all the time they spent with that in Part 1, it had no bearing on Part 2. We get a grumbly Aberforth who is bitter over something the movie doesn't care about.
This film resolidifies my hatred of OOTP, where they cut scenes that would have given the film more emotional resonance in favor of that god-awful Hagrid's half-brother storyline. Not only did they not pay off Grawp in this one, they didn't make up for the completely botched "Snape's worst memory" bit.
I also wanted to echo some of the thoughts regarding the battle. They should've given more screentime to showing our favorite characters battling Death Eaters and the fates of the more recognizable Death Eaters. As others have said, Molly Weasley's duel with Bellatrix was awkwardly staged with the Weasley family essentially just standing on the left side of the frame watching their matriarch fight one of the most vicious Death Eaters around.
What the heck happened to Crabbe? They replaced him with a random black kid (or rather replaced him with Goyle, and replaced Goyle with a random black kid).
They also failed to explain the reappearence of Dean Thomas, who had gone missing (according to the radio reports in Part 1).
I don't recall Lupin and Tonks ever mentioning having a kid, and yet Harry asks the ghost Lupin about him? I imagine there's a deleted scene somewhere of Lupin visiting Harry and the gang at the cottage and taking Luna back to Hogwarts.
They never make the connection between Harry's invisibility cloak being one of the Deathly Hallows, but I guess it doesn't matter anyway all things considered.
Anyone else who would've preferred seeing Bellatrix and Voldemort's lifeless bodies hit the ground rather than them ridiculously exploding into a million pieces or disintegrating?
I also didn't care for the extended Nagini fight and Harry/Voldemort thing. Neville's moment in the book was just much more badass. While I didn't mind Harry and Voldemort's duel, it wasn't as impactful as it should have been. It never felt like Harry was in real danger and then it ended.
Anyway, I enjoyed it overall. I got all teary eyed twice and I'll probably watch it again next week. :)