<!-- 
RSS generated by JIRA (9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66) at Wed Feb 07 21:09:31 UTC 2024

It is possible to restrict the fields that are returned in this document by specifying the 'field' parameter in your request.
For example, to request only the issue key and summary append 'field=key&field=summary' to the URL of your request.
-->
<rss version="0.92" >
<channel>
    <title>MongoDB Jira</title>
    <link>https://jira.mongodb.org</link>
    <description>This file is an XML representation of an issue</description>
    <language>en-us</language>    <build-info>
        <version>9.7.1</version>
        <build-number>970001</build-number>
        <build-date>13-04-2023</build-date>
    </build-info>


<item>
            <title>[CDRIVER-445] int32 and int64 warnings </title>
                <link>https://jira.mongodb.org/browse/CDRIVER-445</link>
                <project id="10030" key="CDRIVER">C Driver</project>
                    <description>&lt;p&gt;I wish to have no more warnings (without simple casting) related to int32 and int64&lt;/p&gt;

&lt;p&gt;So just to make sure don&apos;t forget : &lt;a href=&quot;https://github.com/jeromelebel/libbson/commit/90ea713ff46cf7281c4706d4eb01954aefed766e&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://github.com/jeromelebel/libbson/commit/90ea713ff46cf7281c4706d4eb01954aefed766e&lt;/a&gt;&lt;/p&gt;</description>
                <environment></environment>
        <key id="164294">CDRIVER-445</key>
            <summary>int32 and int64 warnings </summary>
                <type id="4" iconUrl="https://jira.mongodb.org/secure/viewavatar?size=xsmall&amp;avatarId=14710&amp;avatarType=issuetype">Improvement</type>
                                            <priority id="4" iconUrl="https://jira.mongodb.org/images/icons/priorities/minor.svg">Minor - P4</priority>
                        <status id="10038" iconUrl="https://jira.mongodb.org/images/icons/subtask.gif" description="">Backlog</status>
                    <statusCategory id="2" key="new" colorName="default"/>
                                    <resolution id="-1">Unresolved</resolution>
                                        <assignee username="ezra.chung@mongodb.com">Ezra Chung</assignee>
                                    <reporter username="jeromelebel">Jerome Lebel</reporter>
                        <labels>
                    </labels>
                <created>Fri, 17 Oct 2014 23:26:51 +0000</created>
                <updated>Mon, 18 Sep 2023 08:38:47 +0000</updated>
                                            <version> 1.0.2 </version>
                                    <fixVersion>2.0.0</fixVersion>
                                                        <votes>0</votes>
                                    <watches>3</watches>
                                                                                                                <comments>
                            <comment id="788560" author="jeromelebel" created="Wed, 17 Dec 2014 18:19:19 +0000"  >&lt;p&gt;Oh ok. So the issue should not be closed, but set to 2.0&lt;/p&gt;</comment>
                            <comment id="788540" author="jason.carey" created="Wed, 17 Dec 2014 18:06:23 +0000"  >&lt;p&gt;I understand the frustration, and again, apologies.&lt;/p&gt;

&lt;p&gt;I&apos;m in a position where I can&apos;t break ABI at this point, so we&apos;ll have to wait on 2.0 to adjust all the public headers.&lt;/p&gt;

&lt;p&gt;Honestly, being able to kill the 128 byte alignment attribute for bson_t&apos;s is the big thing I want to get rid of.  Being able to run sanitizers sanely would be nice.&lt;/p&gt;

&lt;p&gt;Sorry,&lt;br/&gt;
Jason&lt;/p&gt;</comment>
                            <comment id="788315" author="jeromelebel" created="Wed, 17 Dec 2014 15:59:58 +0000"  >&lt;p&gt;having a 32bit length is not issue : there is already a check to make sure a bson doesn&apos;t grow longer. Even with int32_t we already have the problem of checking the bson size.&lt;/p&gt;

&lt;p&gt;I&apos;ve already done all the changes that needs to be done. MongoHub already works with size_t and ssize_t. Of course I still might have left some bugs, but so far so good. I kept asking for this patch because for everything the more we wait, the harder it will be.&lt;/p&gt;

&lt;p&gt;The idea of my patch was to avoid a lot of cast everywhere.&lt;/p&gt;

&lt;p&gt;But ok, I give up.&lt;/p&gt;</comment>
                            <comment id="788296" author="jason.carey" created="Wed, 17 Dec 2014 15:51:12 +0000"  >&lt;p&gt;Jerome,&lt;/p&gt;

&lt;p&gt;Sorry for the slow turnaround.&lt;/p&gt;

&lt;p&gt;In retrospect I&apos;m not sure why I said I could take this for 1.1.  Pre-2.0, I can&apos;t make any modifications to the public abi, so we&apos;ll have to cut this down to changes in private headers and .c files.  Additionally, we can&apos;t really support a 64 bit api for libbson, simply because the bson specification uses 32 bit lengths internally.  In retrospect, it might have been wiser to use size_t and ssize_t everywhere, but that would just be about enforcing narrowing conversions in libbson at runtime, rather than leaving it to the user.&lt;/p&gt;

&lt;p&gt;Sorry again about not being more prompt on this.  But I can&apos;t really merge the spirit of this patch.&lt;/p&gt;

&lt;p&gt;Regards,&lt;br/&gt;
Jason&lt;/p&gt;
</comment>
                            <comment id="788120" author="jeromelebel" created="Wed, 17 Dec 2014 12:29:09 +0000"  >&lt;p&gt;I guess I have no hope to see one day APIs that support correctly 64bits?&lt;/p&gt;</comment>
                            <comment id="755914" author="jeromelebel" created="Tue, 4 Nov 2014 19:19:39 +0000"  >&lt;p&gt;I have more stuff related to that. I forgot the issue with bson_iter_binary()&lt;/p&gt;

&lt;p&gt;The declaration should be:&lt;br/&gt;
void&lt;br/&gt;
bson_iter_binary (const bson_iter_t  &lt;b&gt;iter,        /&lt;/b&gt; IN */&lt;br/&gt;
                  bson_subtype_t     &lt;b&gt;subtype,     /&lt;/b&gt; OUT */&lt;br/&gt;
                  size_t             &lt;b&gt;binary_len,  /&lt;/b&gt; OUT */&lt;br/&gt;
                  const uint8_t     *&lt;b&gt;binary)      /&lt;/b&gt; OUT */&lt;/p&gt;

&lt;p&gt;Do you need a pull request for that? This difference makes a lot of changes.&lt;/p&gt;</comment>
                            <comment id="755907" author="jason.carey" created="Tue, 4 Nov 2014 19:14:40 +0000"  >&lt;p&gt;good catch.  I&apos;ll fix that for the 1.1 release&lt;/p&gt;</comment>
                    </comments>
                    <attachments>
                    </attachments>
                <subtasks>
                    </subtasks>
                <customfields>
                                                                                                                                                                                                                                                                                                                                                                    <customfield id="customfield_15850" key="com.atlassian.jira.plugins.jira-development-integration-plugin:devsummary">
                        <customfieldname>Development</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                            <customfield id="customfield_10857" key="com.pyxis.greenhopper.jira:gh-epic-link">
                        <customfieldname>Epic Link</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>CDRIVER-4576</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                            <customfield id="customfield_21553" key="com.atlassian.jira.plugin.system.customfieldtypes:labels">
                        <customfieldname>Quarter</customfieldname>
                        <customfieldvalues>
                                        <label>FY24Q4</label>
    
                        </customfieldvalues>
                    </customfield>
                                                                                            <customfield id="customfield_12550" key="com.pyxis.greenhopper.jira:gh-lexo-rank">
                        <customfieldname>Rank</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>2|hr6i7w:</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10558" key="com.pyxis.greenhopper.jira:gh-global-rank">
                        <customfieldname>Rank (Obsolete)</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>1163</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                    <customfield id="customfield_10557" key="com.pyxis.greenhopper.jira:gh-sprint">
                        <customfieldname>Sprint</customfieldname>
                        <customfieldvalues>
                                <customfieldvalue id="304">C Sprint 9</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                            </customfields>
    </item>
</channel>
</rss>