nudles commented on a change in pull request #529: SINGA-484 Code analysis with
LGTM
URL: https://github.com/apache/incubator-singa/pull/529#discussion_r324411358
##########
File path: include/singa/io/transformer.h
##########
@@ -56,12 +56,12 @@ class ImageTransformer: public Transformer {
Tensor Apply(int flag, Tensor& input) override;
- const bool featurewise_center() const { return featurewise_center_; }
- const bool featurewise_std_norm() const { return featurewise_std_norm_; }
- const bool horizontal_mirror() const { return horizontal_mirror_; }
- const int resize_height() const { return resize_height_; }
- const int resize_width() const { return resize_width_; }
- const float rescale() const { return rescale_; }
+ bool featurewise_center() const { return featurewise_center_; }
Review comment:
Does the code analysis suggest to remove the const?
Where can we find all the suggestions?
Thanks.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
