Are you adding support for parameters passed with the shortcode?
e.g. [gallery captions=false] etc. so that features can be changed on a per post basis rather than having to add code to theme files.
Or [gallery id="nn"] where nn is another post with a set of gallery images, which would show the gallery images attached to post "nn" in the current post.
Looking at your code, it looks almost there?