labofsahil/github-event-dataset-2012 huggingface dataset:
The standard size is the size=60.

I know the intend is to break it when the size is specified, but the "problem" is that we have many places in the code where we mention the size (in the Administration for example). All those places will be broken.
Also even if you have the size defined and you let the full width on textarea and select, the aspect will still be broken.
Sorry, see the following commit for an eclipse XWiki code formatted version.
Wrong javadoc, it is not an `hql query`, this method accepts a type.

Typo, should be `parameterized`.
I'd suggest:

childrenRefs.getByName(name) match {
  case None => _actorOf(props, name)
  case _ => throw new InvalidActorNameException("actor name " + name + " is not unique!")
}
The top-levle remarks was around these:

 private val clusterCommandDaemon = system.systemActorOf(
 	 376	
+    Props(new ClusterCommandDaemon(system, this)), "clusterCommand")
 	 377	
+
 	 378	
+  private val clusterGossipDaemon = system.systemActorOf(
 	 379	
+    Props(new ClusterGossipDaemon(system, this)).withRouter(RoundRobinRouter(nrOfGossipDaemons)), "clusterGossip")
Not a python issue, but it would be good to have one longer test with random items and duplicates.
I don't like this. It assumes all tests are doing only mock expectations prior to invoking #run, which is a poor assumption to make.
use config object (see Store comment)
mail-not-confirmed
please store the previous value, and reset its value at the end of the function
For this file, the indent style is not good opening bracket should be on the same line
I think that texturepacker doesn't support file format 3.
Instead, I think that Zwoptex supports file format 3.
Sure? Even if it's only to read the value? I know move semantics makes things cheaper but they keep saying it's still not free. I'll do it if you confirm you prefer by value, I have no problem with that. :)
Easy enough. I'll fix it right up. 

Sent from my digital shackles. 

On Mar 29, 2012, at 8:36 AM, Tim Simpson<reply@reply.github.com> wrote:

>> from reddwarf.common import exception
>> from reddwarf.common import utils
>> from reddwarf.openstack.common import wsgi as openstack_wsgi
>> 
>> -
>> +ContextKey = 'reddwarf.context'
> 
> According to PyLint, as a constant this should be all caps.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/hub-cap/reddwarf_lite/pull/47/files#r621011
Will be in next refresh
Why do you need a protocol version here, when we already send around the Infinispan version?

My worry here is that each command could now start implementing a per-command protocol which a version per command, which would add to the payload.

We have Infinispan version information in the payload header which we can 'switch' on to support multiple payload protocols.
Floor division: http://www.tutorialspoint.com/python/python_basic_operators.htm
I wonder if this stuff should live in mpl-data - given the fact that the ```mpl-data``` folder has a subfolder for ```sample_data```, ```mpl-data``` could very easily have a ```web_static``` directory...
Is this file from somewhere or written from scratch?
Again, can check against the data layer
Use "return !window.navigator.mozWifiManager.enabled"
A future test should be in a separate file, so they can be controlled independently via the manifest. At least finish the call in the setUp then (waiting for the app to be running)
this is a part of the repo and should not be here
this file is the default for the credentials that are used in the test
I just wanted to make a note that though we want it to be an abstract cross right now because  the particle filter isn't in the master, we'll want it to be a yellow cross when that happens. I say that because we're only considering one set of coordinates for goal posts, so it seems like we should only consider one set of coordinates for the cross as well... We should talk about it/how we should determine coordinates with the Particle Filter tomorrow.
copy? compare? hä?
This line should actually be
    $release = $name
Move these `)))` up.
we're using the parent of `$el` in all cases, can you initialize it as the parent instead of selecting it each time?
same here, first SG_REF
Good catch! I forgot about that completely.  
I agree, it's better to remove the second condition so if a column in future will get a description from annotator other than default, the default one on the model will be replaced as well
yeah, locally you can do the following:

git pull --rebase https://github.com/scottmuc/victorykit.git master
I'm setting page->modify->first_id, but I'm not sure that's correct, can you please confirm that?
Ack, done.
Shouldn't this be `Doctrine\ORM\Mapping`?
Is is really version 1.0 of kitchen sink?  Not a big issue but could be updated easily
if title are the same than just add another interval date
@filinep I can't find what you are talking about, I see number of iterations in my version as well as here. Are you referring to line 130?
This is a bad idea IMO. It means the evaluated template cannot use any of the custom filters and functions.
Note to self: add php format.
I'm going to split these tests out very soon so we can avoid the dodgy commas.
There is no toString() method on int, so I use Integer.

Factored out code.
Yes. But I noticed this just by running the code for few minutes. You should also notice it!
Mybe have a problem from string to data convesation (slovenian)..
This should not be needed? `flush_session` should simply remove this key? And this should be enough everywhere that it is false. We should not require that there is explicit disable for mocking in session. If there is nothing in session, it should be disabled.
(As you did.)
Are you sure about that? `and` short circuits.
in last commit, have returned LABEL description to previous state.
This would look less silly if only 1.8 had `#keep_if`.
No Heroku nem precisa disso. Ele sobreescreve esse arquivo lá.
+1 For the 3 line version :)
There is no need to inject the container as you don't use it
yes, it's unneeded, but maybe it could break some tests, so they will have to check the options instead, but let's keep it this way if you don't have time it's not a so big deal ;)
Yes I chose the JDBC driver as the datastax guys told me it's the way of the future. Now it's still using thrift underneath so if we believe that Thrift is much easier to maintain for simple / CRUD operations, I am fine with mixing JDBC and Thrift.

@Sanne , can you give the link to the Thrift based Cassandra CacheLoader so that @jetoile can get a taste of it.
I use `multi` here because the Redis client called [Jedis](https://github.com/xetorthio/jedis) that I use requires us to call `multi`  command when we want values back from Redis within transactions according to the wiki named [Advanced Usage - Transaction](https://github.com/xetorthio/jedis/wiki/AdvancedUsage). I would think that the author of Jedis tries to optimize transactions no matter how many operations are put in a transaction because all the transactions in Jedis are started by the `multi` command as long as I know.

> it looks like more of an optimisation for multiple operations to be batched in a single disk sync in the transactions log.

I do understand the characteristic of the command but I followed the usage of Jedis here.
how about let's validate the $parsed is actually a string here?
and => or
If the 1.1 generated file have these properties, I'd rather go for the generated file then.
This should be removed to avoid confusion.  This master list will be persisted in the database not memory.  I was informed that the most recent version after this will have this list removed.
What say you, @sergi ?
Isn't `site_payload` already a local variable here? It's passed into the function.
Yes we need it has it buil the GUI according to guest capabilities. 

If commented we have to click on the buttonGuest to refresh the GUI (for example, features that are not available appears has switchable until we rebuild the GUI according to capabilities either by the button or by uncommenting those lines above)
doh, yep.
When the function is linear with radius, the enclosed flux is cubic, so there is an analytic inversion but it's got all kinds of square and cube roots to it.  So I thought it would be simpler and maybe even faster just to divide the interval further, since another step down a binary tree is probably quicker than several exp/sqrt/pow function evaluations.

But of course that's just my guess and I'm sure there are ways to make this faster.  The one I'm going to do soon is to make a binary tree search that's optimized for the case when we know how frequently each node will be accessed.  I've already got it figured out but decided to push something that works before trying to push something that's tweaked.  Maybe it would be a good idea to start a new issue / branch for photon-shooting speed improvements as soon as this one is pulled into master.

On May 18, 2012, at 3:41 PM, Mike Jarvis wrote:

>> +            _differentialFlux = integ::int1d(integrand, 
>> +                                             _xLower, _xUpper,
>> +                                             RELATIVE_ERROR,
>> +                                             ABSOLUTE_ERROR);
>> +        } else {
>> +            // Integrate the input function
>> +            _differentialFlux = integ::int1d(*_fluxDensityPtr, 
>> +                                             _xLower, _xUpper,
>> +                                             RELATIVE_ERROR,
>> +                                             ABSOLUTE_ERROR);
>> +        }
>> +        _fluxIsReady = true;
>> +    }
>> +
>> +    // Divide an interval into ones that are sufficiently small.  It's small enough if either
>> +    // (a) The max/min FluxDensity ratio in the interval is small enough, i.e. close to constant, or
> 
> I didn't try to follow all the math here, but might it be better to stop when an interval is nearly linear, rather than nearly constant?  Linear can still be done analytically, right?  And I would think it would be more accurate and faster than trying to get intervals that are piece-wise constant.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/GalSim-developers/GalSim/pull/156/files#r847518
>
Ah, I see, later you also build a lookup table for the x values, hence it becomes "analytic" in the SBProfile sense.
that is the easy fix, the better fix would be to check if carried by raildroid or by player..
Can get rid of = for just a string
good spot, fixed
nullable false?
This should have `.id` due to the reasons mentioned above.
What's the reason for the link being unique?
Because *logically* it is one and larger - it made more sense to me and it is what documentation referred too (and hence that's why it's in the comment too).
o atributo model.is_available é do tipo BooleanField, não seria melhor passar um ``boolean`` ao invés de in ``int``?
Aplicado.
Done in 37f6766
When I click on the name, it throws an error. Test it first with bob@gmail.com
It would be cool for this to configurable but possibly-soon-to-land orientation changes are the right way in the long term.  Should be easy enough to change in a followup anyway.
You don't need to use `DIRECTORY_SEPARATOR` to write portable paths, it's only needed for doing string comparisons to paths from the system.
In my opinion, this is confusing. ```user.name``` is the keypath going to be used for ```-valueForKeyPath:``` with the dictionary, but in the entity, ```name``` is the **key** going to be used for ```setValueForKey:```
Can you fix the whitespace please?
By implementing both JobRegistry and StepRegistry, this really isn't a MapJobRegistry anymore.  It's really a MapJobAndStepRegistry.  This implementation forces me to potentially have two stepRegistries...one embedded in the JobRegistry and one I inject in other parts of the application.  I'd prefer this was not updated unless there's a strong need to for this feature.
Oops, think you mean cinder, not keystone? :)
```
        pathArray = [],
        borderArray = [];
```
Иначе не работает
Зачем это скопировано без изменений?
throw new FormatterException("Type doesn't supported.", new Throwable());
так?
I dunno if this is the most elegant way to get multiple returns from a method.. seems hackish, but it suits the need. Ideas?
Created an issue for this: #31
yes. See the testsuite for instance.

I suggest using ``input`` (and then ``output`` once you implement it)
That small block of logging code is Windows-only because I only needed that low-level logging protocol logging on my windows development machine.
D'oh! Technically unused but bad potential example. Good catch!
same issue with root_path.
no. Thanks :)
I wanted to let @ogrisel give his opinion here, but he seems busy
elsewhere. I think that the best way is to send a pull request. Thanks a
lot!
Have you benchmarked the reservoir sampling implementation? How does it perform vs the set-based implementation CPU-wise?
I'd just set it to EndTime to make sure they're identical. It'd be weird to have then off by a tiny amount :)
Hi Ian.  Is it just me or is this href unnecessarily complicated?  Perhaps I am missing something but it seems like it should just be convert/functions.xsl.
`is a QuadTree type` ?
you refer to Java types? or if of the QuadTree kind ?
I liked it while developing the code because I could easily search/locate the code I was working on, but i guess its time to remove it
It is, but shouldn't, otherwise the switch has no point. @ghalliday I'm assuming the return is the correct behaviour and will remove the break, do you agree?
This smells like a switch... ;) 

Maybe in an utility function of its own, if trying to combine with ```calcIntersectingRowInformation``` above?
this 'n' is the total number of subfiles found. This part of the code is not pretty, I agree.
Looks like both work in the old site, but the link is for builds.rss

I think we should do whatever is more logical and internally consistent for the new site - we can always handle old URLs using Apache rewrites (or similar mechanisms)
Ah, I get it now! OK I adjusted, although note that this test was not failing because "Page Not Found" is present elsewhere
Sure (but it doesn't actually matter to this view, this is the one that the ajax spinner is checking to see if it's available)
Can we get a comment above this indicating the magic that makes the resolver sub-keys available directly in the template?
Do we care whether or not to use British-style phrasing, American-style phrasing, or something else? For example, Americans speak as though an organization is a singular entity, hence "Why _has_ [FRAMEWORK X] not been voted into the group?" Brits speak as though an organization is a pluralized entity.

Not a big deal. Just nitpicking. :)
This is mainly a summary and provides a nice snapshot of information to the user without sending them off to read random markdown files in a GitHub repo. Making a page for the accepted standards is certainly something that needs to be done, but it is not done yet.
*why* should it raise an exception?
@yuhunglin we actually use the difference between date and date_time to differentiate between allday events and not.
See https://github.com/ets-berkeley-edu/calcentral/blob/master/app/assets/javascripts/angular/controllers/upNextController.js

We prob need a different way of doing this now.
Note for server-side reviewer: This feels odd since "getUser" should really be the name of the method that returns a User object. But that method signature is already taken by the HTTP-GET-handler which returns a lot more to the client than just the User data (and also expects local storage to be in place for the person's account). How'd you feel about me changing the GET handler's method name?
dashboard and campus weren't used
~~~clojure
(or (= (sort hand) (range (first hand) (+ (first hand) 5)))
    (= (sort hand) [2 3 4 5 14]))
~~~
Change this to cursor_id since id is a python built-in:

http://docs.python.org/library/functions.html#id
Good catch.  This caused disable_functions to be interpreted incorrectly when it's "'s weren't randomly encoded. 

Fixed in bbfe0f8f495f379cd5f797a47857177237cc82e6
I don't see anything about this technique on that URL.
psexec expects one parameter
Indent size == 4
By default, all the URLs should be exactly the same as they were before. So with default configuration, all existing templates should work. If the URL preferences are changed, then the template should use the correct lookups to get the values.
Why do you still need this? I think it can be removed, because of  DatabaseDriver.php#L306
Are there any non-whitespace changes here?
Such logic is normally in a template, and the controller choose the appropriate template to render. However, we don't have the templating infrastructure  in place (maybe we should get one), we have to do it in such a ugly way.

This logic is essentially converting one view (HTML) into another (XML). Is it good to pass a view into a utility object?
for sake of consistency, let's avoid c-like coding styles, e.g., while(condition) /n {, and use the agreed style: while(condition){
Method name is incorrect. `insertBefore`
"bubbles" should be a constant
Instead of writing:

Function readEntries = () {
      reader.readEntries(...);
}

Why don't you write:

void readEntries() {
   ...
}


On Mon, Nov 5, 2012 at 8:56 PM, adam <notifications@github.com> wrote:

> In web/file/terminal/Terminal.dart:
>
> > +
> > +        StringBuffer html = formatColumns(e);
> > +        for (int i = 0; i<e.length; i++) {
> > +          html.addAll(['<span class="', e[i].isDirectory ? 'folder' : 'file','">', e[i].name, '</span><br>']);
> > +        }
> > +        html.add('</div>');
> > +        writeOutput(html.toString());
> > +      }
> > +    };
> > +
> > +    // Read contents of current working directory. According to spec, need to
> > +    // keep calling readEntries() until length of result array is 0. We're
> > +    // guarenteed the same entry won't be returned again.
> > +    List<Entry> entries = [];
> > +    DirectoryReader reader = cwd.createReader();
> > +    readEntries = () {
>
> Interesting to note, the following does not work.
>
>     Function readEntries = () {
>       reader.readEntries(
>           (List<Entry> results) {
>             if (results.length == 0) {
>               success(entries);
>             } else {
>               entries.addAll(results);
>               readEntries(); // XXX: Not able to resolve closure.
>             }
>           }, errorHandler);
>     };
>
>
> This however does work. I have a feeling in C# the previous does work, I
> could be mistaken.
>
>     Function readEntries;
>     readEntries = () {
>       reader.readEntries(
>           (List<Entry> results) {
>             if (results.length == 0) {
>               success(entries);
>             } else {
>               entries.addAll(results);
>               readEntries();
>             }
>           }, errorHandler);
>     };
>
>
>  —
> Reply to this email directly or view it on GitHub<https://github.com/dart-lang/dart-html5-samples/pull/44/files#r2038934>.
>
>
Seems URI::REGEXP::UNSAFE isnt in 1.9.3 anymore :(
Yeah this is confusing. I will only be recompiling the generated files on merge commits in the future to make reviews easier.
No debug information should be output here. Maybe you can add a protected `_log()` method and turn off output by default
I am sorry but I feel this is not an improvement. I find it confusing to look at compared to the simple if/else. I am sure it can be done better than what I had but I just feel this isn't the best way.
Phasing out in use and examples. It'll stay otherwise. Anyway, it's a rather hackish way to check whether that's the equality predicate - the more direct solution is to just have an `alias`-less overload.
this is ok, there's no need to remove ->utf8
All this logic is useless: you don't need to keep the old service in a different id as you are not using it. the goal of this logic is to use composition.

However, I think using composition here could be far better than using inheritance: using inheritance means that only one customization of the client can be applied at a time.
I think canceling causing a pop-up message is unnecessary and a little wordy, but if you feel strongly about that, let's chat about it.
don't comment; remove you've an scm
Any reason why this couldn't be an attribute? Maybe a user wants to fork all the repos and use their forks instead?
I didnt even know this existed. go community!
It seems like if it is on the notifications page, it should be saved with the notification settings in the config...
I will look at the other changes or someone else can take care of it.
Attach an id/class to where the notification should show up
put the service in the ``security.xml`` file. No need to create a new file each time.
Why pass an empty string here? I'd expect something similar to what you have in the test above.
remove trailing command for 2.6.x compat
setListeners do not transform the value it receives.

in `RegisterEventListenersAndSubscribersPass`, `$listeners` look like this (ordered by priority):

```
array(
    array($eventData['event'], array($id, $method), $class, $format),
)
```

Whereas `EventDispatcher\LazyEventDispatcher::$listeners` look like:

```
array(
    'event' => array(
        array(array('id', 'method'), 'type', 'format'),
        array($callable, 'type', 'format'),
        array(array('id', 'method'), 'type', 'format'),
    ),
)
```
All the variables are suffixed by `!default` so that we can override them before the file is imported.
good call.  fixed.
Add space between `){`
I'd move this onto one line: `self.assertEqual(self.model.CaseVersion.objects.count(), 0)`
I don't think this should to be an API property, it's for internal use only.
Do you think we could expose these prefixed with `thread_`? I suspect I might need them at some point when writing my virtual machine, since I need to be able to directly mutate the stack (as dirty as that sounds).
Actually looking at your code again you probably want to provide a custom message
```
Assert.isInstanceOf(ExtendedSession.class, session, "mget not supported by...");
```
Oh! No!
Gary, thanks: I understood what you mean about **LIFO**.
So, I see two ways:
1. Add check about *startup indicator* here too and in the `reschedulePersistedMessagesOnStartup()` invokes `this.releaseMessageAfterDelay(message, delay);` as scheduled task to avoid initialization thread deadlock.
2. Or add some additional method to MS, which will be similar to `removeMessageFromGroup()`, but don't return MG, when message wasn't deleted by current transaction or return some boolean with the same logic. In this case I just can iterate over MG's messages.
WDYT?
@ghillert I don't understand the need for this attribute at all; why can't we always use the message as the parameter source; they would simply have to prefix each property with payload. From what I can tell, it delegates to a bean wrapper to traverse the properties.

I would think the "fix" here would be to remove this from the schema rather than adding this complexity.

I don't think we give the choice between message a payload as the root for a SpEL expression. so why would we do that here (which is similar).
Tuple is a new type we will be adding.

The only reason I suggest limiting to Map is that it's trivial to get a List from a Map (map.values()) if someone really wants that, and furthermore - where draw the line... why not support an array?

My main concern is that a List is only useful to someone who knows what each index points to, and the Tuple type will provide a better structure for that.
True, won't deny it. Leftovers from design by code. . . anyway its fixed.
Was there a reason that this file_line approach was not working for you?  I was trying to avoid having to manage the entire file, because I'm concerned about compatibility between postgres 8.x and 9.x.
How about renaming this to something like "local_user_management_add_users_fileedit"? (not completely convinced about the name) The idea would be to allow the existence of something like "local_user_management_add_users_systemtools", that does the same thing but through the user-management system tools instead of direct file edits.
汎用例外クラスだと思いますので、適切な場所に移してもらえますでしょうか。
Why must their default be false? Just let it be null would be enough I guess
@bp323 Why do we need the `typeof` in here, doesn't `$.isFunction` suffice?

http://jsfiddle.net/Jtr4p/
Yes, I will reword that. Both are being deployed so really all they have to do is modify index.html and redeploy and it will work
Thanks for the help on the regex, I'll add these in the next commit.
We're moving this sort of cleaning to filters. This is OK for now, but eventually we'll have to remove the newline stuff.
`exists; pulling changes` ?
This change is part of the root-enabled-history changes.
Using instances.get because of the then-missing mgmt API; when I make the root enabled history available as a standalone mgmt call, I'll change these tests again to reflect that.
instead of converting empty arrays, we could also check for ``false`` without allowing type casts
Since this is just a single view, it should be added as a single view, not using `include`.
Also we don't need the namespace, since we don't need to reverse it anywhere inside REST framework.

eg.

url(r'^auth-token/', 'rest_framework.authtoken.obtain_auth_token')
ok thanks
cheers,

Javi



On Thu, Aug 16, 2012 at 12:49 PM, Romain Deltour
<notifications@github.com>wrote:

> In pom.xml:
>
> > @@ -591,5 +632,15 @@
> >        <name>Eclipse Gemini Maven repository</name>
> >        <url>http://download.eclipse.org/gemini/mvn/</url>
> >      </repository>
> > +    <repository>
>
> Yes, ideally. Note that I'm not strongly against using 3d party bundle
> repo, but on the other hand having our own let us control our OSGi props
> and avoid potential reliability issues with no-longer maintained 3d party
> repo (which I doubt will happen anytime soon with Spring's repo though).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/daisy-consortium/pipeline-assembly/pull/1/files#r1391672>.
>
>
@alindeman Good point. So we can delete the is_a?(Hash) check.
I think this one should be moved in the page header (`#content > h1`). The first `h1` is in a `<header role="banner">`, so I assumed it’s ok to keep the website title only here, whatever the current page is.

The problem is that the second one is located in several templates (maybe we could use a wrapper function everytime we need a page title, something like `lesintegristes_page_title("My original title")`).

Tell me if you agree, or not :-)
To be honest I would just put `s=100` in the URL for now rather than faffing around with options like that, given that we're not really using them.
remove all these parameters, vibrate, sound, ledFlash!!
"whether to recursively create any missing parent directories in the given path."
"...should return ... an InputStream that can be used to retrieve the contents of the file."
So it would make sense to keep the global configuration, as the default.
Then for each collection we would do

```
/*
 * @Hateoas\Relation("...",
 *     ...
 *     content = {
 *          ...
 *          "pager" = {
 *              "pageParameterName" = "index",
 *              "limitParameterName" = "maxPerPage"
 *          }
 *          ...
 *     }
 * )
 */
```
In my experience, using arbitrary short sleeps less than a few seconds is not sufficient and, increasing the sleep time, unnecessarily increases the build time, which can be frustrating. I prefer to use a "smart" sleep; most of the spring-integration-ip tests have something like this...

````java
	private void waitListening(TcpInboundGateway gateway) throws Exception {
		int n = 0;
		while (!gateway.isListening()) {
			Thread.sleep(100);
			if (n++ > 100) {
				throw new Exception("Gateway failed to listen");
			}
		}
		
	}
````

which waits for up to 10 seconds buts exits as soon as we are ready.
So is there more than one id? Still don't get what the id os for.

If you do have multiple event streams you may want to look into writing something like the eventemitter in node 
------Original Message------
From: Rowell Cruz
To: Gord Tanner
Subject: Re: [webplatform-framework] Updating webview api to a more generalized factory (#11)
Sent: Mar 12, 2012 3:11 PM

> + * Unless required by applicable law or agreed to in writing, software
> + * distributed under the License is distributed on an "AS IS" BASIS,
> + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> + * See the License for the specific language governing permissions and
> + * limitations under the License.
> + */
> +
> +var Webview = require("./webview"),
> +    utils = require("../utils"),
> +    events = require("../events"),
> +    chrome = require("../chrome"),
> +    _webviews = [],
> +    _self;
> +
> +// Setup listeners
> +events.on(0x610BA11D, "Created", function (something) {

We were trying to come up with a global id that we can subscribe to.  We would fire an event on that id for every event we cared about, which is Created and Destroyed.  Not the best in terms of performance not that I think about it.  Any thoughts on how to have global events?

--- 
Reply to this email directly or view it on GitHub:
https://github.com/blackberry-webworks/webplatform-framework/pull/11/files#r548104

Sent on the TELUS Mobility network with BlackBerry
Я этой кнопкой пользуюсь редко (разве что найти нужный тред в первых пяти страницах). Сейчас начал играться и только один раз меню не исчезло, при этом я скроллил страницу. Мб дело в этом?
what happens if smtpServerDetails returns nil?

As could be the case, looking at https://github.com/quicksilver/com.apple.Mail-qsplugin/pull/1/files#L175R92

I just tried `[nil objectForKey:@"something"]` in the debugger, and it gave `Target does not respond to this message selector.`
If I get to this stage, I'll try debugging it and update this comment thread. If there's no update, then... sorry :(
No ... it's definitely not. Better solution needed (like the TODO says)
what is this line for? it doesn't seem to be an api
Cool. Let me know if you need any help. I have worked in the config-parser file a lot :).
seems like there's room for refactoring here. looks like duplicated logic to me.