<p></p>
<blockquote>
<p dir="auto">Since types must be declared before use, members that use public types need to go after that definition, and if the type is public and the data member private, its a common idiom to put those data members in one private section at the end of the class declaration rather than swapping back and forth all the time.</p>
</blockquote>
<p dir="auto">Once again - the "ignore variables after the current line" thing is for <em>local variables only</em> (that is, variables declared inside a single function). It doesn't apply to members. Anyway, no problem to remove it. As <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/kugel-/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/kugel-">@kugel-</a> pointed out, the possible problem could be when refactoring/writing a not yet compiled code and having the variable declared after the current location by accident. I'm still a bit hesitant about how common problem this is and if we should support invalid code (which we don't already, a single extra <code>{</code> breaks ctags parsing). But OK, let's remove this condition.</p>
<pre><code>         if(p){           // autocompletes p first but with lots of C library crap
...
                pit = 1;         // autocompletes with ctrl space after p 
                                 // but with pid_t before pit in list
...
</code></pre>
<p dir="auto">This is normal autocompletion (scope autocompletion only happens when you type <code>.</code> or <code>-></code>) where the result contains all tags.</p>
<pre><code> foo();  // no tooltip for foo(
</code></pre>
<p dir="auto">I haven't done anything regarding tooltips. Tooltips right now just look at corresponding function calls which <code>foo</code> isn't. Could probably be improved.</p>
<blockquote>
<p dir="auto">As noted there are some places where the sorting seems questionable.</p>
</blockquote>
<p dir="auto">The sorting is alphabetical, like in the symbol tree (capitals first). I haven't done anything regarding the sorting of the results and I don't really have any smart idea how to (simply) guess what the user wants.</p>
<blockquote>
<p dir="auto">Also constructors, Foo() above, should not show in . or -> completions lists (p-> in main() above), they can't be called like that. Destructors can and show ok.</p>
</blockquote>
<p dir="auto">Constructors are reported by ctags as methods so we see them in the same way as methods. We could probably detect them by checking if the "function" name is the same as the name of the class and filter them out.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/3175#issuecomment-1107815201">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ3OUV4ONAN75IEDW2TVGUROLANCNFSM5T5CX5RA">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAIOWJ5WCBMCTYOCXONEQGDVGUROLA5CNFSM5T5CX5RKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOIID6WII.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><geany/geany/pull/3175/c1107815201</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/3175#issuecomment-1107815201",
"url": "https://github.com/geany/geany/pull/3175#issuecomment-1107815201",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>