why is this code here and in QgsGdalProvider::srcDataType? It would be better to have a single method that does this. e.g. QgsGdalProvider::bandType(int bandNo)