-
Notifications
You must be signed in to change notification settings - Fork 592
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove Slice-to-C++ code generation for local Slice #1619
Conversation
// | ||
// Ice version 3.7.10 | ||
// | ||
// <auto-generated> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This code is the old generated code. We need to clean it up in a follow-up PR.
Ordered | ||
} | ||
/// Uniquely identifies TCP endpoints. | ||
const short TCPEndpointType = 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The EndpointSelectionType enum is now in the EndpointSelectionType.ice file - a more appropriate name!
@@ -20,7 +20,9 @@ | |||
|
|||
#include <Ice/ObjectAdapterF.ice> | |||
#include <Ice/ConnectionF.ice> | |||
#include <Ice/Context.ice> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both Context and OperationMode are non-local so can't be in the same file as local struct Current.
// Copyright (c) ZeroC, Inc. All rights reserved. | ||
// | ||
|
||
#pragma once |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Split IceBox.ice in Service.ice (local) and ServiceManager.ice (non-local).
da61c2e
to
158364b
Compare
c7f9bf9
to
fcfc762
Compare
This PR removes the Slice-to-C++ code generated for local Slice definitions. With this update, slice2cpp no longer generates anything for local Slice.
This PR:
commits the generated code for local C++ definitions, usually as-is (to simplify reviews)
split a few shared Slice files (in ice/slice) that included both local and non-local definitions
See #1620.