From 8deaec1531fa996306e756c3a6b4a3bd0394dc9d Mon Sep 17 00:00:00 2001
From: Priit Pender <priit@alloca.ee>
Date: Mon, 18 Mar 2024 13:39:12 +0200
Subject: [PATCH] Fix code review issues.

---
 main.cc             | 3 ++-
 sar/az_multilook.cu | 4 +++-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/main.cc b/main.cc
index ba79646..2eb1da2 100644
--- a/main.cc
+++ b/main.cc
@@ -352,7 +352,7 @@ int main(int argc, char* argv[]) {
             lvl1_file_handle.WriteSync(mds.buf, mds.n_records * mds.record_size);
             LOGI << "IMP done @ " << lvl1_out_full_path;
         }
-        if (alus::asar::specification::IsSLCProduct(asar_meta.target_product_type)) {
+        else if (alus::asar::specification::IsSLCProduct(asar_meta.target_product_type)) {
             // az_compressed_image is unused from now on.
             constexpr size_t record_header_bytes = 12 + 1 + 4;
             const auto mds_record_size = subsetted_raster.XSize() * sizeof(IQ16) + record_header_bytes;
@@ -415,6 +415,7 @@ int main(int argc, char* argv[]) {
             lvl1_file_handle.WriteSync(mds.buf, mds.n_records * mds.record_size);
             TimeStop(file_write, "LVL1 file write");
         } else {
+            throw std::runtime_error("Unknown/unsupported output product type");
         }
     } catch (const boost::program_options::error& e) {
         ExceptionMessagePrint(e);
diff --git a/sar/az_multilook.cu b/sar/az_multilook.cu
index 1a36aa0..dc73ac1 100644
--- a/sar/az_multilook.cu
+++ b/sar/az_multilook.cu
@@ -1,8 +1,10 @@
 #include "az_multilook.cuh"
 
-#include "alus_log.h"
 #include "checks.h"
 
+// TODO implement frequency domain multilooking? Is is better
+
+// Implements sliding window azimuth time domain multilooking and azimuth resampling in one step
 __global__ void TimeDomainAzimuthLook(const cufftComplex* data_in, int x_size, int in_y_size, int out_y_size,
                                       float resample_ratio, float* data_out) {
     const int x = threadIdx.x + blockDim.x * blockIdx.x;