[jira] [Resolved] (PROTON-2751) Reduce use of pn_buffer_t when not needed

2023-07-11 Thread Andrew Stitcher (Jira)


 [ 
https://issues.apache.org/jira/browse/PROTON-2751?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Stitcher resolved PROTON-2751.
-
Fix Version/s: proton-c-0.40.0
   Resolution: Fixed

> Reduce use of pn_buffer_t when not needed
> -
>
> Key: PROTON-2751
> URL: https://issues.apache.org/jira/browse/PROTON-2751
> Project: Qpid Proton
>  Issue Type: Improvement
>  Components: proton-c
>Reporter: Andrew Stitcher
>Assignee: Andrew Stitcher
>Priority: Major
> Fix For: proton-c-0.40.0
>
>
> pn_buffer_t has complex logic to hold and manage a ring buffer. But is being 
> used just to manage a resizeable bytes buffer. And in most of those places 
> the buffer doesn't really nned to be resized either!
> So it can be replaced in many places with much simpler code.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Resolved] (PROTON-2753) Split util.h to separate the C++ safe parts out of the C99 but not C++ parts

2023-07-11 Thread Andrew Stitcher (Jira)


 [ 
https://issues.apache.org/jira/browse/PROTON-2753?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Stitcher resolved PROTON-2753.
-
Fix Version/s: proton-c-0.40.0
   Resolution: Fixed

> Split util.h to separate the C++ safe parts out of the C99 but not C++ parts
> 
>
> Key: PROTON-2753
> URL: https://issues.apache.org/jira/browse/PROTON-2753
> Project: Qpid Proton
>  Issue Type: Improvement
>  Components: proton-c
>Reporter: Andrew Stitcher
>Assignee: Andrew Stitcher
>Priority: Major
> Fix For: proton-c-0.40.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Resolved] (PROTON-2752) Use pn_string_t/pn_bytes_t instead of raw char*

2023-07-11 Thread Andrew Stitcher (Jira)


 [ 
https://issues.apache.org/jira/browse/PROTON-2752?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Stitcher resolved PROTON-2752.
-
Fix Version/s: proton-c-0.40.0
   Resolution: Fixed

> Use pn_string_t/pn_bytes_t instead of raw char*
> ---
>
> Key: PROTON-2752
> URL: https://issues.apache.org/jira/browse/PROTON-2752
> Project: Qpid Proton
>  Issue Type: Improvement
>  Components: proton-c
>Reporter: Andrew Stitcher
>Assignee: Andrew Stitcher
>Priority: Major
> Fix For: proton-c-0.40.0
>
>
> There are a few places where we hold a pn_string_t convert it to char* then 
> convert it to pn_bytes_t. We can "cut out the middleman"! and be more 
> efficient and have less code overall!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (PROTON-2750) [protonj2] Add some additional functionality to the AMQP test peer

2023-07-11 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/PROTON-2750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17742163#comment-17742163
 ] 

ASF subversion and git services commented on PROTON-2750:
-

Commit 19e48d4fa31e67fd8a70746f081906b39083d959 in qpid-protonj2's branch 
refs/heads/main from Timothy Bish
[ https://gitbox.apache.org/repos/asf?p=qpid-protonj2.git;h=19e48d4f ]

PROTON-2750 Ensure the frame parser is ready for AMQP header on connect

Always reset the frame parser to expect AMQP headers on a new connection

> [protonj2] Add some additional functionality to the AMQP test peer
> --
>
> Key: PROTON-2750
> URL: https://issues.apache.org/jira/browse/PROTON-2750
> Project: Qpid Proton
>  Issue Type: Task
>  Components: protonj2
>Affects Versions: protonj2-1.0.0-M16
>Reporter: Timothy A. Bish
>Assignee: Timothy A. Bish
>Priority: Major
> Fix For: protonj2-1.0.0-M17
>
>
> Add some additional testing APIs to the test peer.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (PROTON-2750) [protonj2] Add some additional functionality to the AMQP test peer

2023-07-11 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/PROTON-2750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17742164#comment-17742164
 ] 

ASF subversion and git services commented on PROTON-2750:
-

Commit 06341414b44ca796cd7d4fa09da83bc66920be26 in qpid-protonj2's branch 
refs/heads/main from Timothy Bish
[ https://gitbox.apache.org/repos/asf?p=qpid-protonj2.git;h=06341414 ]

PROTON-2750 Add API for expecting null source or target in Attach

Allows easier scripting of expecting null source or target in Attach
response which would inidcate incoming detach from remote.

> [protonj2] Add some additional functionality to the AMQP test peer
> --
>
> Key: PROTON-2750
> URL: https://issues.apache.org/jira/browse/PROTON-2750
> Project: Qpid Proton
>  Issue Type: Task
>  Components: protonj2
>Affects Versions: protonj2-1.0.0-M16
>Reporter: Timothy A. Bish
>Assignee: Timothy A. Bish
>Priority: Major
> Fix For: protonj2-1.0.0-M17
>
>
> Add some additional testing APIs to the test peer.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (PROTON-2751) Reduce use of pn_buffer_t when not needed

2023-07-11 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/PROTON-2751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17742157#comment-17742157
 ] 

ASF subversion and git services commented on PROTON-2751:
-

Commit a2c54b3a245356b734e22bbdd5f732ed00d44398 in qpid-proton's branch 
refs/heads/main from Andrew Stitcher
[ https://gitbox.apache.org/repos/asf?p=qpid-proton.git;h=a2c54b3a2 ]

PROTON-2751: Use pn_rwbytes_t not pn_buffer_t for scratch space


> Reduce use of pn_buffer_t when not needed
> -
>
> Key: PROTON-2751
> URL: https://issues.apache.org/jira/browse/PROTON-2751
> Project: Qpid Proton
>  Issue Type: Improvement
>  Components: proton-c
>Reporter: Andrew Stitcher
>Assignee: Andrew Stitcher
>Priority: Major
>
> pn_buffer_t has complex logic to hold and manage a ring buffer. But is being 
> used just to manage a resizeable bytes buffer. And in most of those places 
> the buffer doesn't really nned to be resized either!
> So it can be replaced in many places with much simpler code.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (PROTON-2752) Use pn_string_t/pn_bytes_t instead of raw char*

2023-07-11 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/PROTON-2752?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17742160#comment-17742160
 ] 

ASF subversion and git services commented on PROTON-2752:
-

Commit 97fcd648fd826a68b3e0d681d054c11031e6b06a in qpid-proton's branch 
refs/heads/main from Andrew Stitcher
[ https://gitbox.apache.org/repos/asf?p=qpid-proton.git;h=97fcd648f ]

PROTON-2752: Make more use of pn_string_t instead of char*


> Use pn_string_t/pn_bytes_t instead of raw char*
> ---
>
> Key: PROTON-2752
> URL: https://issues.apache.org/jira/browse/PROTON-2752
> Project: Qpid Proton
>  Issue Type: Improvement
>  Components: proton-c
>Reporter: Andrew Stitcher
>Assignee: Andrew Stitcher
>Priority: Major
>
> There are a few places where we hold a pn_string_t convert it to char* then 
> convert it to pn_bytes_t. We can "cut out the middleman"! and be more 
> efficient and have less code overall!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (PROTON-2751) Reduce use of pn_buffer_t when not needed

2023-07-11 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/PROTON-2751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17742158#comment-17742158
 ] 

ASF subversion and git services commented on PROTON-2751:
-

Commit f2693f29d1226b224477c3eab62654c49499ec6b in qpid-proton's branch 
refs/heads/main from Andrew Stitcher
[ https://gitbox.apache.org/repos/asf?p=qpid-proton.git;h=f2693f29d ]

PROTON-2751: Don't use pn_buffer for delivery tag


> Reduce use of pn_buffer_t when not needed
> -
>
> Key: PROTON-2751
> URL: https://issues.apache.org/jira/browse/PROTON-2751
> Project: Qpid Proton
>  Issue Type: Improvement
>  Components: proton-c
>Reporter: Andrew Stitcher
>Assignee: Andrew Stitcher
>Priority: Major
>
> pn_buffer_t has complex logic to hold and manage a ring buffer. But is being 
> used just to manage a resizeable bytes buffer. And in most of those places 
> the buffer doesn't really nned to be resized either!
> So it can be replaced in many places with much simpler code.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (PROTON-2751) Reduce use of pn_buffer_t when not needed

2023-07-11 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/PROTON-2751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17742159#comment-17742159
 ] 

ASF subversion and git services commented on PROTON-2751:
-

Commit 31e25638ea74551465f1331c41bdfaced5438b69 in qpid-proton's branch 
refs/heads/main from Andrew Stitcher
[ https://gitbox.apache.org/repos/asf?p=qpid-proton.git;h=31e25638e ]

PROTON-2751: Remove unused functionality of pn_buffer_t

Also rearrange implementation struct to be more efficient


> Reduce use of pn_buffer_t when not needed
> -
>
> Key: PROTON-2751
> URL: https://issues.apache.org/jira/browse/PROTON-2751
> Project: Qpid Proton
>  Issue Type: Improvement
>  Components: proton-c
>Reporter: Andrew Stitcher
>Assignee: Andrew Stitcher
>Priority: Major
>
> pn_buffer_t has complex logic to hold and manage a ring buffer. But is being 
> used just to manage a resizeable bytes buffer. And in most of those places 
> the buffer doesn't really nned to be resized either!
> So it can be replaced in many places with much simpler code.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Commented] (PROTON-2753) Split util.h to separate the C++ safe parts out of the C99 but not C++ parts

2023-07-11 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/PROTON-2753?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17742156#comment-17742156
 ] 

ASF subversion and git services commented on PROTON-2753:
-

Commit 0545b51c234260fe6977e8545c785a442732950d in qpid-proton's branch 
refs/heads/main from Andrew Stitcher
[ https://gitbox.apache.org/repos/asf?p=qpid-proton.git;h=0545b51c2 ]

PROTON-2753: Split util.h so that we can keep the C++ in the schannel impl happy


> Split util.h to separate the C++ safe parts out of the C99 but not C++ parts
> 
>
> Key: PROTON-2753
> URL: https://issues.apache.org/jira/browse/PROTON-2753
> Project: Qpid Proton
>  Issue Type: Improvement
>  Components: proton-c
>Reporter: Andrew Stitcher
>Assignee: Andrew Stitcher
>Priority: Major
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (PROTON-2753) Split util.h to separate the C++ safe parts out of the C99 but not C++ parts

2023-07-11 Thread Andrew Stitcher (Jira)
Andrew Stitcher created PROTON-2753:
---

 Summary: Split util.h to separate the C++ safe parts out of the 
C99 but not C++ parts
 Key: PROTON-2753
 URL: https://issues.apache.org/jira/browse/PROTON-2753
 Project: Qpid Proton
  Issue Type: Improvement
  Components: proton-c
Reporter: Andrew Stitcher
Assignee: Andrew Stitcher






--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (PROTON-2752) Use pn_string_t/pn_bytes_t instead of raw char*

2023-07-11 Thread Andrew Stitcher (Jira)
Andrew Stitcher created PROTON-2752:
---

 Summary: Use pn_string_t/pn_bytes_t instead of raw char*
 Key: PROTON-2752
 URL: https://issues.apache.org/jira/browse/PROTON-2752
 Project: Qpid Proton
  Issue Type: Improvement
  Components: proton-c
Reporter: Andrew Stitcher
Assignee: Andrew Stitcher


There are a few places where we hold a pn_string_t convert it to char* then 
convert it to pn_bytes_t. We can "cut out the middleman"! and be more efficient 
and have less code overall!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org



[jira] [Created] (PROTON-2751) Reduce use of pn_buffer_t when not needed

2023-07-11 Thread Andrew Stitcher (Jira)
Andrew Stitcher created PROTON-2751:
---

 Summary: Reduce use of pn_buffer_t when not needed
 Key: PROTON-2751
 URL: https://issues.apache.org/jira/browse/PROTON-2751
 Project: Qpid Proton
  Issue Type: Improvement
  Components: proton-c
Reporter: Andrew Stitcher
Assignee: Andrew Stitcher


pn_buffer_t has complex logic to hold and manage a ring buffer. But is being 
used just to manage a resizeable bytes buffer. And in most of those places the 
buffer doesn't really nned to be resized either!
So it can be replaced in many places with much simpler code.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org