[
https://issues.apache.org/jira/browse/AVRO-3051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17307097#comment-17307097
]
Thiruvalluvan M. G. edited comment on AVRO-3051 at 3/23/21, 2:06 PM:
---------------------------------------------------------------------
[Here|https://github.com/apache/avro/pull/1153/files?w=1] is the third
installment. [~emkornfi...@gmail.com] is it possible to review? For some
reason, I am not able to add your id as reviewer in the Pull Request. When
viewing the diff, to ignore trivial white-space-only changes, please add
{{?w=1}} to the URL.
was (Author: thiru_mg):
[Here|https://github.com/apache/avro/pull/1153/files?w=1] is the third
installment. [~emkornfi...@gmail.com] is it possible to review? For some
reason, I am not able to add your id as reviewer in the Pull Request.
> Modernize C++ implementation
> ----------------------------
>
> Key: AVRO-3051
> URL: https://issues.apache.org/jira/browse/AVRO-3051
> Project: Apache Avro
> Issue Type: Improvement
> Components: c++
> Reporter: Thiruvalluvan M. G.
> Assignee: Thiruvalluvan M. G.
> Priority: Major
>
> We have mandated C++11 in out builds. But the code itself is mostly C++98 or
> C++03 style. Thanks to C++ backward compatibility, it works well. But moving
> to "modern C++" (in the sense used by [Scott
> Meyers|[https://learning.oreilly.com/library/view/effective-modern-c/9781491908419/],]
> will make the code more compact, more readable and in some cases even faster.
> In this ticket I intend to introduce a series of pull requests modernizing
> the C++ implementation. I'll take extra care to keep the interface to the
> users the same. So most of the changes will be in the {{impl}} directory. The
> {{api}} directory may change a bit but will not break the current clients.
> For the initial round, we'll continue make it compile against C++11 standard.
> We can reap even more benefits by moving to something like C++17. But that is
> for another day.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
