Message ID | 20191011140408.11712-1-laurent.pinchart@ideasonboard.com |
---|---|
State | Accepted |
Commit | 8647991cd8310463d1f5d08698f277e2278e77d9 |
Headers | show |
Series |
|
Related | show |
Hi Laurent, Thanks for your patch. On 2019-10-11 17:04:08 +0300, Laurent Pinchart wrote: > Use const references in a for loop to avoid an unnecessary copy. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > src/ipa/rkisp1/rkisp1.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp > index 4f5fdb146212..02419e254f2d 100644 > --- a/src/ipa/rkisp1/rkisp1.cpp > +++ b/src/ipa/rkisp1/rkisp1.cpp > @@ -98,7 +98,7 @@ void IPARkISP1::configure(const std::map<unsigned int, IPAStream> &streamConfig, > > void IPARkISP1::mapBuffers(const std::vector<IPABuffer> &buffers) > { > - for (IPABuffer buffer : buffers) { > + for (const IPABuffer &buffer : buffers) { d00h, Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se> > bufferInfo_[buffer.id] = buffer.memory; > bufferInfo_[buffer.id].planes()[0].mem(); > } > -- > Regards, > > Laurent Pinchart >
diff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp index 4f5fdb146212..02419e254f2d 100644 --- a/src/ipa/rkisp1/rkisp1.cpp +++ b/src/ipa/rkisp1/rkisp1.cpp @@ -98,7 +98,7 @@ void IPARkISP1::configure(const std::map<unsigned int, IPAStream> &streamConfig, void IPARkISP1::mapBuffers(const std::vector<IPABuffer> &buffers) { - for (IPABuffer buffer : buffers) { + for (const IPABuffer &buffer : buffers) { bufferInfo_[buffer.id] = buffer.memory; bufferInfo_[buffer.id].planes()[0].mem(); }
Use const references in a for loop to avoid an unnecessary copy. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- src/ipa/rkisp1/rkisp1.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)