labofsahil/github-event-dataset-2012 huggingface dataset:
You know, you can set this value in the userInfo dictionary:

mappedKeyName: chestDropRate.name

for a single value....but still, the to-one and to-many code is already there....not sure why it's missing that even though you changed the import to update...

On Feb 24, 2012, at 12:13 AM, Brian Doig wrote:

> "chestDropRate" : [
>    {
>      "name" : "foo",
>      "sequence" : 0,
>      "value" : 0
>    }
>  ]
> 
> works but
> 
> "chestDropRate" : {
>      "name" : "foo",
>      "sequence" : 0,
>      "value" : 0
>    }
> 
> throws an exception because it tries to iterate over the keys of a NSDictionary which returns NSStrings which then throw a selector not recognized.
> 
> On Feb 23, 2012, at 3:09 PM, Magical Panda Software wrote:
> 
>> Can you give me an example of how this crashes? I'd ideally like to make a test case for this, but I'm also curious since this hasn't crashed here for me in a while...
>> 
>> --- 
>> Reply to this email directly or view it on GitHub:
>> https://github.com/BrianDoig/MagicalRecord/commit/32cf9d23363e137917b1ce89c5f85af336bc09e5#commitcomment-1004870
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/BrianDoig/MagicalRecord/commit/32cf9d23363e137917b1ce89c5f85af336bc09e5#commitcomment-1004883
that might complicate the method sig but you could reuse the same array and use an index to represent the virtual start point
Good idea. Still figuring out Java.

Maybe some sort of warning method we can pass an exception to?
http://wow.joystiq.com/2009/10/27/ptr-patch-3-3-patch-notes-updated/6

better?
DELETE FROM `command` WHERE `name` = 'reload waypoint_data';
INSERT INTO `command` (`name`, `security`, `help`) VALUES ('reload waypoint_data', 3, 'Syntax: .reload waypoint_data will reload waypoint_data table.');
Hello burnham

men have some mail

or skype to send some things in private?

I need your help thanks
I'll do something like that. Thanks for the tip.
But i'll try to write some macros. It's to complicated to have ARC otherwise
Fixed now for default locale
It does, but not sure if it is flexible enough for you. If you include a link it will hyperlink to it (i.e. if it starts with something like http...). If the link is to an image then it will make it an image tag. Not sure how it determines if it is an image. Probably by looking at the extension. Does Travis end with the proper extension or does it have the extension neutered?
Hi, you are right I just wanted to smuggle in something which I added to my configuration. So take that only as proposition. I would like to have also some conceal for "not is" and "is" but I cannot find appropriate characters.
I apologize for my previous three commits, the first was accidental and the last two were attempts to undo what it changed.
Notice in the foreach we use the OfType method provided by the IEnumerable API. Very nice helper method.
why push-4-bytes for the height:  I ran a tool that looked at previous coinbases, and no-opcode turned up plausible heights, as did the push-3-bytes-opcode.  push-4 was completely clean for all existing coinbases.
What if we'd just skip this check? The user would get an `AttributeError`, which seems appropriate enough.
Yes, I think this should work, but the original purpose of this patch was to maintain availability of all cached eggs, even those which have been removed from official pypi.python.org server.
BTW, I'm still interested to commit these patchs upstream in the collective svn. (see: http://www.mail-archive.com/plone-product-developers@lists.plone.org/msg00094.html)
That's odd - if it works fine, then why are you trying to patch a problem?  Isn't that the definition of "not working fine"?

Removing soft lockup detection and panic on soft lockups is going to result in the phone ending up in oddball states and make debugging issues extremely hard.  Things should never be locking up in the first place.

If you look in /proc/last_kmsg after an LPM crash, you'll likely see that kinteractived is the task that hung.
Part | Points
------------- | -------------
base | 1
adv |  0
*Total* | 1

Passes tests. :sparkles:  missing adv section.
Do we want to keep the "action" class on this, and add "action-deleterecord" as an additional class, to assist with CSS styling?
Sounds nice. I'll be looking forward to test when I get my new device. Thanks for all the effort!
I added the application environment to the MoM namespace because it was useful to disable the print popup for integration testing (Selenium and other browser drivers do not handle popups well, and they are even worse at OS native dialogs like the print dialog, so it's best to just avoid those). This could also be useful in the future for other javascript tests, otherwise I would not have added it. Let me know if this seems like we're trying too hard to accomodate integration testing - if so we can just ditch the tests.
Haha... that's funny - I made a similar change shortly after you... after I realized it was a bug.  Glad we were on the same wavelength.
rsstail programı kurulu olmalıdır. BlendOs için :
$ sudo yaourt -S rsstail
ubuntu için :
$sudo apt-get install rsstail
why won't it ever work?
I'm just using these wonderful greyscale values I got used to many years ago. I never liked HEX and never will, thus I always use rgb() values for debugging purpose.
hmm... this is essential, i dont know why its not refreshing when adding empty directories
Return an `Iterator<ClientRecord>` or an immutable `Map<String, ClientRecord>` instead. No sense returning a concrete mutable collection.
it's a typedef to the function pointer so later in ofMap the argument type can be specified as ofEasingFunction instead of the whole function pointer. also it'll make cleaner the autocomplete
This breadcrumbs definition doesn't match the `Hierarchy::getBreadcrumbs` function that Ingo made.  Would it be realistic to define a single data structure for breadcrumbs across the whole of SilverStripe?  Perhaps we could even have a `SS_BreadcrumbList` object with a `forTemplate()` method that performs the concatenation?
Maybe just drop the if clause an set it on each call to `fetchPage()`. That way the timestamp would always be up-to-date. Something I could do for the Ruby implementation, too.
今フラグを利用しないようにするには、このオブジェクトがロードされた時に、accepted_answerがnilのときという判定がいいのでは？

accepted_answer_id_was というような、初期値は何だったかというメソッドがあるのでそれを使えばいいかなと思います。
Die Beschreibung stimmt nicht mit dem comand überein, Danke das du es alles in blage umbenennst :-)
Remove readwrite and nonatomic.
Hmm, works on my... and i got the new driver as you...
Probably rom code... i am on cm7 latest nightly from source.
missing a paranthasis
Surely this would be nicer with a class?
--___--
but i always do it if i only want to build only application or modules
never got this error before.
ok, maybe i will try to full build it tommorow
So, let me make sure I get this right: you just rebind the existing  Backbone.sync to Backbone.ajaxSync so that it can be used somewhere else?
This seems fair :) LMK if I'm missing something. Otherwise I'll merge!
Make a guess about what this does, use your knowledge of the = operator and your experience with $0 from previous exercises.
move this after the permission check
ich glaube diese werte sollten auch funktionieren:

+#define  S5PV210_VIDEO_SAMSUNG_MEMSIZE_FIMC0 (5632 * SZ_1K)
 	 
+#define  S5PV210_VIDEO_SAMSUNG_MEMSIZE_FIMC1 (4608 * SZ_1K)
 	 
+#define  S5PV210_VIDEO_SAMSUNG_MEMSIZE_FIMC2 (6144 * SZ_1K)
 	 
+#define  S5PV210_VIDEO_SAMSUNG_MEMSIZE_MFC0 (32768 * SZ_1K)
 	 
+#define  S5PV210_VIDEO_SAMSUNG_MEMSIZE_MFC1 (32768 * SZ_1K)

bei meinen bisherigen kerneln hat keiner probleme damit gemeldet.
I was remembering this commit: 8abfde4 which looks like a switch to using `clientTop` and `clientLeft` which gets around the IE `documentElement` border issue.
Default value `None` is no problem. It silently ignores formal parameters that don't exist in the definition of the function being decorated and actual arguments when the validation is called. Documentation fix on the way.
Arguably compile and handle methods don't make sense on a compile-time (rather than runtime) kernel object
Well then technically self should be an option or status should be an Option, if there can exist such a state where it doesn't exist.
hmmm, should we allow that? Ideally a sendSystemMessage failure should put it on resend-duty for the "guaranteed delivery" of system messages, right?
use an akka.util.Deadline instead: https://github.com/akka/akka/blob/master/akka-actor/src/main/scala/akka/util/Duration.scala#L259
Wow, what a shame. Then I suggest we push this feature in to 2.1 instead of 2.0.2 since we can't make it non-kludgy while retaining binary compat.
Shouldn't getChild be only in UntypedActorContext? It's purpose is for java api.
For OSGi purposes, can we place this in akka.jmx package instead?
what are you getting at?
cool. good to know this too. that's nice.
Is that a typo? "_get_shm_ceof"?
`if followee is not None` would be more Pythonic
Doesn't the binding need to indicate which distributor type is being bound too?
I wonder if it is worth consolidating these methods so there isn't so much repetition? They are each nearly identical. It looks like the only difference is the action tag and the manager method.
Back ticks, not asterisks for this formatting.
I'll reword.
why is this one file removed?
Call the new rule.
@elupus: I fixed the above, as you can see :-) Thanks for the heads-up, I didn't realise how we handle zip/rar paths etc.
Then we might as well do that all over the place. It doesn't make sense to handle possible single-stepping/debugging this way IMO.
Where I can read about coding style you using in this project?
I think eventually the wrapper script will have to be a shell script anyway so that people can use `export` in their setup commands.

And let's just call it `interpreter`. :)
Not sure what you mean? Do you mean in the comments? It's own class?
Need to check if the bin op is one of the supported ops here
Är inte van vid att ha versions nummer i kommentaren så jag gömde att uppdatera det.
I nästa uppdatering jag gjorde på forken kom jag dock i håg det.
Same reply as above :)
This file is auto-generated from the i18n file downloaded from cucumber so adding stuff in it is wrong.
You're not losing it, but apparently I did.  Somewhere that function was left behind.
Please change to create_listing() {
scala.concurrent
Hmm, alright. I don't find this elegant, but it works without ambiguity.
I'll patch that in tomorrow.
Can you remove those two examples? I think we should be good to go after that.
You don't need to say -actionsForTarget... any more.
This should be `NSString * const TUITextRendererDidBecomeFirstResponder = @"TUITextRendererDidBecomeFirstResponder";`
You call this function with always the same parameters. And IMHO, there's no need to returns an array.

Your function should just check if mainconfig exists, or if defaultconfig exists, and should returns only the full path of the found file. or null if it doesn't exists.
One, this is unreadable. Two, what do you think happens when you call `uv_read2()` on a stream type that doesn't support it?
Yes. If you know a better formulation, let me know.
これもいいねー
You expect to receive "PONG" from Redis when you send the command "PING"; if there was a connection problem (i.e. no response), this would throw an exception. "FAILED TO PING" means that the command didn't return True ("PONG"). But this is really to test the connection so we should never have this case.
я не могу писать комментарии к комментарияем. это не к этому относилось
закомментированный код надо либо убрать, либо прописать, зачем он тут
Maybe! I'm not sure. I figured it was done the old way for a reason, and was trying to make as small a change as possible.
put:

```
Warn when the `:locals` option is passed to `assert_template` outside of a view test case
Please take a look at https://github.com/rails/rails/pull/8127#discussion_r2083275 :)
`kind_of?` is better than using `.class ==` here.
Agree. Thanks
Thanks Santiago <3
In commit 834a8faa123f168e8baea772b06ebce2874ff431 I moved the adaptation of types of return expressions from `addAnnotations` to `typedReturn` via the `adaptTypeOfReturn` hook, which is a new hook I added to `AnnotationCheckers`.

In commit cd14c3087ceca8145c3013c2751648b936f1256d I also improved the doc comment on `adaptTypeOfReturn` of `CPSAnnotationCheckers` to explain exactly when the type of a return is adapted:

    /** Returns an adapted type for a return expression if the method's result type (pt) is a CPS type.
     *  Otherwise, it returns the `default` type (`typedReturn` passes `NothingClass.tpe`).
     *  
     *  A return expression in a method that has a CPS result type is an error unless the return
     *  is in tail position. Therefore, we are making sure that only the types of return expressions
     *  are adapted which will either be removed, or lead to an error.
     */
Looks good.  Mind if I add a CityHash implementation in here?
Oh look, already an illustration of why I said to separate the error handling and the logic.  Anytime you find yourself doing a blind get on an Option, it's because you should have been able to call a method which returned T, not Option[T], but you couldn't.
This section is too long imho (keep in mind this is reference documentation). The details on slicing notation can be found in the numpy.mgrid docstring already.  Giving a reference with page number should never be done, because it's particular to the pdf version of a particular numpy reference guide.
I didn't mean about the `flash` notice. I meant about the `@order.adjustments.promotion.eligible` check. This is being done twice within this method.,
I think you should ask for integer, rather than real. Also, you should use exp and base rather than args[n].
Are you sure?

http://www.python.org/dev/peps/pep-0008/#other-recommendations

Seems like spaces around these operators is preferred.
Too much typing...
Some nitpicking: The latter half can fit in one line no?
I'll take the space out here and around the "*" then commit this.
in my experience, mixing single and double quotes in a docstring generates sphinx errors...let's see if this happens with your request.
Actually, this comment can be deleted :)
The placement of the following two functions `IntConst_to_Symbol` and `dsolve`  is odd. They are duplicated across two files and seem like they might have functionality that should be in a normal (not test) file.
Yes, splitting it up was pretty easy.  I have moved out the three branches of the huge `if`-statement  into separate functions.  The resulting functions are still relatively long: 1.5--2 screenful of text, but I believe that does not critically affect readability because of numerous chunks of repetitive code.
Let's clean out any extraneous msgs out of anything that is ready for release.
no problem. i wrote this test a long time ago so i figured it'd be ok to fix my own formatting ;)
-1. If you are receiving errors, let's address the root problem instead of treating the symptom. The question is: how is `removeFromBucket` getting called when the item is not in the bucket?

<img src="http://bottomlesswit.com/wp-content/uploads/2012/01/mrBUCKET.gif">
Wow, seems a bit excessive. What's it for?
Please don't touch the abstract interface.
Create your own implementation instead
It's not all that relevant, by the way - we're dropping the libev dependency.
I suppose so. It doesn't make sense for stream types like `uv_tty_t`, though.
Arn't you repeting the same action as in line 62?
So file parameter would get the quotes twice?
This regexp is probably already used somewhere else right?  (to_param and from_param)  Can we just store the regexp as a class constant and use it from two places (I don't want to change one one day and not remember to change the other).  Ditto for the other regexps if they are used in multiple places.
Is there a way to parameterize this test, such that you can the following test data points:

* (ascii collation, downstream query)
* (ascii collation, upstream query)
* (unicode collation, downstream query)
* (unicode collation, upstream query)

?
blank line before this line
As the function is backwards compatible to 3.0 I dont really want to do that
Why is character set none? If utf8 is used in directory name the column must have utf8 also.
Not fixed?
User object.
Path info can have backslashes.
I'm not sure about sending a pm for such information. I think it should be done via email, like it is done when posts are approved and reports are handled.
https://github.com/phpbb/phpbb3/pull/865/files#L3R757 It is defined as config_var and it is initialized once the user enters a data_path through the ACP. So if data_path and config_path have not been entered by the user, config file contents are not generated in the ACP.
Why was this empty line removed?
this line fixes a global leak
Mergea "can't" :P.
Is this going to C++ code style?
I make this to free Light children definitions from long  : public Foo, public Bar, public WhatEver lines. Technically its wrong and should be improved.
I couldn't spot any difference between having standard there and not? :S
This does not seem to be accessed from anywhere
could cache length here. probably not a big deal in this case but it's a good habit.
The refactoring into a constant is a good idea.  The rest is a new feature.
Твой ответ изчез из-за обновления, отвечу. Да, нельзя никому доверять и нужно обязательно делать секьюрную загрузку картинок. Да, сейчас есть дыра в том, что можно указать любой код в html баннере(хотя наверно так и останется), но этом ты не повредишь самой баннерке.
What are the changes in this file? Looks like you've upgraded to VS2012? Did you also change it to .NET 4.5? Either way this is sorted in master already
long line - can we wrap these?
This line is too long. And the same applies to `__str__()`.
This should be in the third-party section.
The check is in there. The diff is just showing it as deleted because the new grad method is in a different location than the old grad method.
Does this mean batched adhoc is always multi-partition?
A pair of parenthesis around this line may make reading easier.
What if two users invoke this view concurrently?
append space to #
Don't use names which are builtins. You might want to call ``list()`` later on and it would result in a non-obvious error message. ``result`` is a fine name for a structure you want to return from a function.
Really nice, readable, well documented and commented code. I wish I wrote like this.
pretty sure you mean `orelse` and `andalso` here and not `or` and `and`
So we could leave it as `.php`.
That's  little vauge. What you're saying seems more broad than what I was thinking:

```
if self.setting('autoAssignDummyTransactionToSelf'):
    self.addChunk('return None')
else:
    self.addChunk('return _dummyTrans and trans.response().getvalue() or ""')
```
Minor issue: Why not just make this :class:`openquake.db.models.BCRDistributionData` so it is a proper link?
jQuery has `jQuery.Callbacks`, is this also exposed publicly?
Oh, I see.  I thought other modules would have the table definition of the result in the "about" page, but it seems no modules have ones. Now it's clear.
Good suggestions. Thanks.
Add an additional function _btrim_ before lower, and add the corresponding comments. Thanks.
Since we're not changing mobile_ua_re and we're treating it like a "constant", we should probably name it MOBILE_UA_RE.
Do we have to do this over and over again as each item is saved? Does scrapy not provide any kind of signal for "ok, all done with everything" that we can hook into and just do this once?
right, I think it's more useful to know that it got interrupted and didn't ever finish
I'm not sure it's possible to reliably test the remctl installation outside of a kerberized environment, I'll use -v for now. Thanks.
Would you mind adding a comment with a link to the upstream issue and an explanation of why this was added?
I added the resume feature because I had huge problems downloadiing the ADHD dataset. Problem is that the resume feature has not been fully tested for the moment and may lead to unexpected results so I have chosen to keep it disabled for the moment. Plus, if we decide to enable it by default later, I think that this won't disturb the user.
This method blurs the line between the DAO and the Verification frameworks.  Please discuss with Chuck whether or not this method should exist.
Why not just `return self.wiki_override` (or, if it *must* be a bool, `return bool(self.wiki_override)`?
Cool.  I'm still trying to suss out how to run all your unit tests locally.
这个理由可以接受。既然到了那种奇葩的地步，那已经不是我这里计划涵盖的内容了……
stylisitically this should be a parts.length.times do |i| ... end loop.
nitpick: I like linebreaks after these short circuit returns to set them off a little
Change to ``Prompt the user to select files or folders with operating system's file open dialog``
Question: Is the relativeID something like the parent? I know we're not yet implementing sub menus, but maybe the API should have something like a parentID, even if those are top-level parents for now?
This line stores the autoconnect status in the session storage.
here is a race condition. The env is essentially global. So any new ide will overwrite workspaceId and basePath for all others.

You should make filelist a constructor and `new` it.
All headers *could* be aligned to 8-bytes, if we rounded up the size of each counting bloom's counters region to a multiple of 8 bytes, but we don't do that yet. On x86 it probably doesn't matter, but it might on mips or powerpc, and just seems nice...
why the if when simply ```return services.contains(service)``` works?
Is there a reason to add this instead of allowing options to have `options[:required]`? Isn't `required` just another option in  way?
Yes, I though about that. But then, it means the call before throwing the exception is useless, since it gets called when caught.
Done. (Presuming "import json,.." is ok?)
This is added automatically.
On the contrary I like it. providing just the regex is like providing half of the puzzle and asking the user to do the rest. Their code is going to look very like this because the nature of the regex suggests the code solution, so why make them do the extra work? Bidpom can be a fully working solution for browserid rather than just half of it.
Perhaps we can do some more work to make it not need values passed through and I notice now perhaps it should be a property.
Rather than all of this preparation, is it possible to just put a special block of html in `trackevent-editor-defaults` that's specifically for start/end? It looks like there's a lot of loopy-logic that happens here (e.g. `startEndCount`) to get this done, and there's a lot of code here just to apply things in the right order and apply correct class names. Instead, why not add a method to `trackevent-editor` that's something like `createStartEndInputs`. Happily, `attachStartEndHandler` then has a much more obvious use-case. :) <-- happy
don't make functions in a loop.
It's this common css block I'm talking about above.
Extra space at the end here.
remove this whole block. there are no longer iframes anymore
This portion shall be exposed as a separate piece of functionality via API, e.g. getAnnotationOperatorList().
[Docs:](http://docs.python.org/3/library/ssl.html#ssl.SSLContext.wrap_socket) (last paragraph)
"Specifying server_hostname will raise a ValueError if the OpenSSL library doesn’t have support for it (that is, if HAS_SNI is False)."
Shouldn't this be `~> 2.0.0`? `~> 2` would allow 3.0 as well
from 1.20 to 1.3?
might be nice to even have a `bm tags` cmd
Heh? What was it? Don't get it. But this should not hurt, ACK :-)
Initially per our offline chat, I was going to add the order to the call in the version(env) method; however, decided to place it in the in_environment(env) method to ensure consistency across calls.
This seems authorization related - should it be in the authorization tests instead?
Out of curiosity, why does this not nil check against a the string "nil" ?
Since we're doing a model now, you can just do:

search = Pool.search do
  blah blah blah, all the same stuff.
end
Please fix the indentation. We use 4 spaces per level, not tabs or 2 space
That sounds broken to me since these types will never change how they are deserialzied.
This is a patch. That does not mean you become an author.
Are you expecting to have more than the PAGE_SCREEN and TABS_SCREEN? If not then using numbers seems a bit overkill?

Using 3 and 4 as a continuation of the URL button modes also seems a bit odd...
Oh, I missed your other comment from the same time, so I now see how this applies .  See my comments below for more, but I definitely agree that the approach @iguananaut suggests here is better than the current scheme.
The transform can be and should be cached.
As before, I think these should be uppercase.
If @laanwj has a strong opposition to "Messaging", then perhaps "Signatures" or "Contracts" (to stress the importance of being careful what you sign) would work?
wrong for expired entries
Please remove it. A file containing a class definition should contain only the class
Sure.  I added it, Thanks!
Ah nevermind, I misread the docstring (it could stand to be clarified). The intention of this widget is to embed an Enaml Container inside a non Enaml ui.
I think the last two functions should be joined into one.
I think I added it when I was fiddling around with escaping strings and forgot to remove it afterwards. I still need better escaping though. Any idea?
creating this monitor socket should probably be inside the `if self.max_heartbeat_misses > 0` block.
I don't think this should get an alias.   Remember, everything is always available at the command-line, even without aliases as:

    $> ipython notebook --NotebookApp.notebook_manager_class='mymod.MyNotebookManager'
Adrian Cole <reply
+p-901028-1a3432458dddecbd20c30bcf5ee042ba0aef38d6-1453210@reply.github.com>
 wrote on 30/05/2012 07:00:06 PM:

> From: Adrian Cole <reply
>
+p-901028-1a3432458dddecbd20c30bcf5ee042ba0aef38d6-1453210@reply.github.com>
> To: Kenneth Nagin/Haifa/IBM@IBMIL,
> Date: 30/05/2012 07:00 PM
> Subject: Re: [jclouds] Added container handling methods and tests (#654)
>
> >   * @see <a href="http://www.snia.org/cdmi">api doc</a>
> >   */
> >  @Timeout(duration = 180, timeUnit = TimeUnit.SECONDS)
> >  public interface ContainerClient {
> > -
> > -   Container getContainer(String containerName);
> > +   Container createContainer(String
> containerName,CreateContainerOptions... options);
>
> please be consistent with indentation
>
Sorry about that.  The inconsistent indentation was not visible to me
because of a mix in spacing and tabs.
I formatted the source with eclipse. Hopefully, that will fix the problem.
-K
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jclouds/jclouds/pull/654/files#r901028
>
What's the rationale for using time_t? Why not Date?

Plus "time_t" is a bit cyptic here. I'm assuming you mean time_t in C/C++ like http://www.cplusplus.com/reference/clibrary/ctime/time_t/

Please make this more clear.
Marked as @/Nullable, but builder checks that it is not null
I've changed it to pass thourgh the error for both `git_reference_resolve` and `git_object_lookup`. (Maybe something we want to make a policy for all wrapper functions.)

> Shouldn't a reference pointing to a non existing object be considered as corrupted?

Dunno, not the scope of this PR.

For loose references we only check if they contain a valid hex object identifier (`loose_parse_oid`), whereas for packed objects we have more certainty the object (still) exists,
`write_json_from_dict` ?
It is already there no? '(r'', include('privacy.urls')),'
I moved this section out of an if block and thus over two spaces.
i would use the `.border-radius(10px)` mixin for this.
Yeah, they're both valid, I just always thought this was cleaner to look at.
`... By default the timezone will be set to "UTC" but check that it's set`.
It at least needs to match both a space and a colon after the address, which are the usual cases we're seeing. Not sure why we'd see an email address there though (and then this wouldn't make things bad, we'd probably be happy it would be masked out anyhow).
Please describe why you chose 120. Why not 150? Or 90?
And a comment why doing this at all.
Whitespace stripping would be appreciated
So why is the length 0 if it is null and not 1? Where and how is "nullness" encoded?
I'm a little confused with how this instruction is worded.
First check if a new configuration value, ``uris/script``, exists, before falling back to the old behaviour of guessing the uri. 

Before this, it was not possible no change which url was used to reference the generated main output file in a "source" build (it was always possible to change it in a release build.
Can we move this file to doc/MULTICONNECTION_README?
That bit was taken from the BMP decoder which has the same styling problem. I'll fix it
we don't add this to test classes
What if someone is using the global temp folder and that contains sessions from other system users that are under different uid/gid?
Should be silenced?
Just to be clear, you're saying to use the existing model's data when there is missing submitted data?

Let's say the existing model has an `Author`.  The form will have something like `<option name="form[author]" value="1" selected />`.  If we did a partial bind and omitted `&author=1`, we would have to do the equivalent of:

```php
$author = $this->config->getData()->getAuthor();
$authorId = doSomeTransformOnTheEntity($author);

$child->bind($authorId);
```
In short, reproduce the `setData` part as if the user actually provided the transformed, submitted data?
Offset handling is not fixed as mentioned below.
And replacing a property by an index (as this method does) would suggest to me, that I can simply omit the name. Otherwise it's not only replacing the type. So having an optional name makes total sense to me. Currently you would need to first fetch the name and then set it again.
you should use the short class name
I said you should change the context to ``js`` for the autoescaping. If you only apply ``|e('js')``, you are escaping as JS but the autoescaping will still escape it as HTML after that as the context is set to HTML by default
See http://twig.sensiolabs.org/doc/tags/autoescape.html
@bschussek My point is as @Tobion mentioned. There is IMO no real point to put variable in class when is used once for one `if` and never touched again.
they won't. We need to register a custom error_handler inside the testcase to catch the deprecated warning before PHPUnit (and restoring the error handler at the end and when an exception occurs)
http://php.net/manual/en/class.badfunctioncallexception.php says "Exception thrown if a callback refers to an undefined function or if some arguments are missing." This seems exactly what is happening in that code, no? Why go for InvalidArgumentException instead? (they are both LogicException's)
same as for _create_pidfile above
cut and paste errors.
If that's the case, we're going to have problems anyway because I'm pretty sure Ubuntu 12.04 switched to Ruby 1.9.
That model seems fragile, because if it happens for some reason that a second value is inserted in the column (migration, restoring an old DB and "merging" things, etc), then you will always get back the bad value. The result should be sorted with descending values to be sure to get the highest index.
I'm not a fan of this if statement. I cannot extend it without writing more code. Could you do some kind of dictionary like thing (maybe even just a dictionary...)
When you remove the software overhead properly you will see it is probably ~5.3MHash
(issues like USB I/O clearly play a part in it)
What does the special device_owner prefix "network:" mean?
That looks like an old version of the code :-(
This message needs to be localised (see org.jboss.as.weld.WeldLogger)
Same here - a WARN message instead of a deployment failure.
Will Thomas' change in https://github.com/jbossas/jboss-as/pull/3425 allow this to be typed?
If the jmx subsystem is installed but the services not yet this could fail?
We are replacing the platform mbean server on startup though so perhaps you could use that instead if you don't want a real dependency.
I would rename this to CdiAnnotationMarker. 

In other subsystems this sort of marker is used to actual signal that a deployment should happen, which is not the case here.
this is the line I refer to in my other comment
Everett has commented that this is causing some issues refocussing the browser chrome when the dialog and screen reader are active. Most likely the whole chunk will go.
I've reported all needs such as these and others upstream. Some he plans to fix, others might take a volunteer to do it.

The person responded positively and with a little hope he might even take my offer of getting it up at GitHub so volunteers can actually do that.
это дебаг закомментиный. заюзал, когда проблему с processTitle обнаружил
nah, leave it if you want, it's ok for me if we all know is a mock :)
If the refactor is for case names and easier readability, I'd like (if possible) this part separated into different lines, so it's easier to check which ones are included:
'''
domSelectors: [
  'splash-screen',
  'progress-bar',
  'activation-screen',
  'finish-screen',
  'tutorial-screen',
  'nav-bar',
  'main-title',
  'pincode-screen',
  'pin-input',
  'sim-import-button',
  'done-button',
  'networks',
  'join-button',
  'date-configuration',
  'time-configuration',
  'date-configuration-label',
  'time-configuration-label',
  'data-connection-switch',
  'fake-sim-pin',
  'pin-error',
  'button-lets-go',
  'button-skip',
  'time-form',
  'sim-import-feedback',
  'refresh-button'
],
'''
Please `typeof evt.retryCount` is indeed a string...
do we absolutely need a `<div>` here? I won’t block on this but I’d prefer if we could live without unnecessary containers.
The `data-items` attribute is purely arbitrary. Let’s not rely on that, I’m pretty sure we can rely on flex properties instead.
On 21/06/2012 04:09, Timothy Guan-tin Chien wrote:
>> @@ -91,8 +95,10 @@ var BackgroundServiceManager = (function bsm() {
>>       if (!app || !hasBackgroundPermission(app))
>>         return false;
>>   
>> -    if (frames[origin] && frames[origin][name])
> `framesFromOrigin` I meant. It's longer than `frames[origin]`!

That's really not a matter of size but not having to access 4 times the 
same object property :)
req.onsuccess = function onsuccess() {
would it be sufficient to change this to an "onclick" ? (Which is widely used across other apps)
I know very little about CSP and how it would effect this.
Here seems a good time to load it.
There is no dependencies between these blocks, but I'm agree that they could share some css living together.
My only concern is having a huge CSS, that is what we don't want.
this is very pretty.
Why reimplementing Pimple instead of adding it as a dependency ?
This should be `URI::regexp`, as we discovered in the other pull request, to support Ruby 1.8.7
@filipewl, I don't found use of the selector. Removed. 0eb93cb224
So it attempts to match and if that fails, it then drops to any that match all and pops one off and returns it?
I've dropped most of the code, since they're now handled more cleanly in the high-level, all-in-one, consistency function ``v4toit.get_standard_consistency()``.
Indexes are a key/value combination. Perhaps if you think of values as rows?
`protected` methods and variables should not be prefixed with `_`.
Hmm.. This code is hard to follow -- can we structure it differently:

    if (!feed) { ... } else { ... }

Also: as the parameter can contain array, we should cast it to be array and test in_array(0, $array)
I think i find the source of the problem.

In the render thread, when we apply the movement, we take the transform and put the result in trasnformlive.
So in the next cycle, if the engine don't make the copy, we erase the previous trannformlive with a new movement always computed with the transform.

I will test to confirm and make a PR.
Honestly though, I'm not sure this needs to be tested, period.  String.replaceAll() is a core java function, and we're guaranteed its going to work as advertised.  The filename cleaner simply does `String.replaceAll( \\W", "_"), which amounts to doing a regex search and replace on anything that does **not** match this pattern: `[^A-Za-z0-9_]` and converting it to an underscore.

Code doesn't get much simpler than this.  Every one of the test cases in `FilenameCleanerTest` are 100% going to pass, unless java breaks a core String library function :)
This has to change really, we can print a warning the first time this occurs in the process, but we can't shut out these users. Not everyone can enable openssl.
if __name__ == "__main__":
 .....
Why product.path but not '.'?
Why is the response 404 and not 403?
@dubcanada it will re restored to origin
You might want to just use: string.printable for this
Nah, if
(1) it is not the first build
(2) the previous one was aborted
(3) and the previous one was aborted before this test case execution (no result for this test case)
start searching the first failure earlier.
Why the `/`?
Don't worry strongly about this, but you might find it easier to read/write this if you split each of these into separate methods -

```
if @integration == "tcp"
  sendMessageTCP(@batch .... )
elsif ...
```
My bad ... what editor/IDE is useful for ruby?
Can't you combine these two conditions ?
See my comment in querystring, can you use `serialize`?
Good catch! It was just completely broken on fennec ...
I don't think page-mod unit test ever run on mobile as I think that they rely on tabs API.

Otherwise, I manually tested and it should be fennec-compatible now.
@lloyd - the "body" tag in "common.css" line 20 declares the same font face universally.
why do you put in localhost?
This file should probably have the "right" extension for the type of content: I would suggest `facts.yaml`, since "cache" is kind of implicit.
You probably want to use `ref`, rather than type and title, here - it better matches the standard way we output resource details.
style-wise this is all way off the rest of the project, try and copy what the rest is like. For example there's no need to have the `clean` function in scope since it's not enclosing anything, and things like `if (test.failed == true` can just be `if (test.failed`, `} else {` vs `}else{` etc
I belive that this one is somewhere round 75 ;-)
wrong initial slash
Concetto datato sin dalla `2.0.0-beta4`
hmm, if request URI has wrong port number, this regex from Uri won't match and port will be set to default
I guess it's better to set port as is and let Uri raise an exception
These should not be skipped.
Hint: Instead of changing the whole nutrition of all the ItemFoods that are of this type. Rather change the value in the FoodLevelChangeEvent on line 42 to the new nutrition value. This way everything still works the eating animation is still fired, the nutrition value is changed only for this one item/time and People can modify the modified nutrition value in FoodLevelChangeEvent. Sorry if i sounded like a smart ass but I could really do with an event like this :).
Indentation.  Fit code to 80 cols please.
i want to refactor these but maybe not the time yet, you are compressing things a bit but it is cool, just was trying to aim at readability
That `now?!!` Don't make sense...
Space after the `:`, applies in many places.
ISTM that this will introduce a compatibility breakage with the current behaviour, no? IOW, users will need to specify FEED_DOMAIN = SITEURL to keep the current behaviour. We can instead rather default FEED_DOMAIN to SITEURL in the settings module (in read_settings). No?
for breadcrumb translation (get $lang if node breadcrumb is a context or a module name
You can read about it here: https://developers.google.com/speed/docs/best-practices/rendering#SpecifyImageDimensions
Remove this todo, since RestValue is inside Common :)
Already in the copyright header
object or super()
I know, I'm writing some unit tests for DI config that will cover such & similar cases.
backticks around class names
That's always the case with creating an error in a deferred callback, I think they're playing with something called domains to try to fix it - but that's not stable yet and I have no idea how it works. For now, having the outer node in the stack is better than nothing at all (at least then you can see where the error originated from and go from there).
Use NULL, not zero. Signals to the reader it's a pointer.
This isn't my code; I want to be able to update this as easily as possible so I don't want to touch the whitespace
Indeed, ignore me.  In general I want us to use this clean-up pattern when it makes sense, though.
PEP8 is complaining about the tab here. Lines with no content should be empty (not even whitespace). It's a pain because most editors add this whitespace.
何故かgit rmしてたので復活・・・
Use `&&`, not `and`.
jesus i wish there was a better way to write this — but this works
Should/could have a class-level @Produces/@Consumes
That's just another whitespace deletion...
Looks like this was always broken.. apparently (perhaps since the beginning of time) if you pressed undo 2 more times than there were things to undo, it would start throwing javascript errors.
Could add our `@Immutable` annotation here.
I think he's subscribing to the belief system that he should only change one thing at a time, aka one file. I think it's admirable, and I'm pulling this into my private Wordpress repo.
Indenting is funny here
The purpose is to wait for the delta to be empty. This means `update_for_names()` updates _only_ for those names and not other changes that were in the delta.
Nitpick: semantically, the logo should probably be displayed in an img element not as a background
this is a bad idea IMO. PSR0 is not a subset of PSR1. So if I ask for PSR1 filters, it should not run the PSR0 fixers
I'm not so sure if I agree, but I'll let it pass.
Is there any other means of querying the page list? `by_destination` seems to imply there are other options, but there really isn't.
The `socialmediabox` section really needs to be cleaned up.

* Remove the references to `managedinternetpresence.com` from the `<img>` elements.
* Please properly indent and perhaps consider that you are **Li**sting * where IMH is on social media...
* Remove the `target` attribute please from the `<a>` elements.
No big; it's a pretty minor nit and a fair amount of C-style code omits optional braces. Just try to include them in the future.
Do we want to use require instead?  I don't think it'll make a difference to the volo optimization step, but we should check either way.
What I meant about the sticky bit was: "why would postgres set it that way?"  I will go with 775 unless you have any ideas what they were trying to do with that.
Is there specific markup we can leverage for examples? I saw the `<pre>` tag being suggested a lot.
Makes sense. Lets use the current solution for now.
Not a big deal, but there is already a concrete implementation class in the Eclipse API: `Region` :)
You can make this fit over three lines, two for the text, 1 for the params.
Since I am afraid of having tons of memory leak or seg fault if we use pointers too much...
ik zou dit hernoemen naar `spawnEnemies()`
Right, now I see why you've dropped the IDisposable interface. Carry on.
'migration_type' here as well :)
Гораздо легче написать `@property`, чем вручную создавать iVar и геттер.
Да, я именно про эту возможность сделать свойство доступным только на чтение.  

--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev


среда, 31 октября 2012 г. в 13:00, hmmvot написал:

> In DNAProject/Cell.m:
> > + NSString *descr = [[NSString alloc] init]; // хочу, чтобы всё выводилось в одну строку! > + for (id elem in DNA) { > + descr = [descr stringByAppendingString:elem]; > + } > + return descr; > +} > + > +-(int)hammingDistance:(Cell *)someCell { > + int ham = 0; > + /* долго думал, и пришёл к выводу, что так сравнивать лучше всего, так как > + у нас нет прямого доступа к массиву DNA объекта someCell */ > + NSString *sourceString = [[NSString alloc] initWithString:[self description]]; > + NSString *compareString = [[NSString alloc] initWithString:[someCell description]]; > + for (int i=0; i<capacityOfDNA; i++) { > + if ([sourceString characterAtIndex:i]!=[compareString characterAtIndex:i]) ham++; > + }  
> 1) Практический смысл в том, чтобы избавиться от не нужной здесь операции преобразования массива в строку.
> 2) @property (https://github.com/property) (nonatomic, readonly) NSArray *DNA; благодаря такому property вы сможете обращаться к DNA из вне, но не сможете изменить указатель, хранящейся в DNA, и не сможете изменить массив, хранящийся в памяти по этому указателю. Вот вам поддержка идеологии объекта.  
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/Hexlet/osx-project-1/pull/17/files#r1989523).  
>  
>  
>
Расширение лучше вынести в отдельный файл, для этого предусмотрена даже команда в меню
Спасибо, пофиксил
It's really hard for me to follow the logic here :\ Variable names are too long as well.
I got this accidently checked in. I need this for QuitApplication().
`NSRunLoop` isn't thread-safe, so this may be unsafe if done from a background thread. To be correct, we should dispatch to the main thread first, or else use `CFRunLoop` and `CFRunLoopTimer` APIs.
This is just `distinctUntilChanged` generalized. I don't see why we'd want to keep it confined to `<RACSignal>`.
Well they don't need this or the previous one exactly, and actually they can use the more normal schedulers, but they do need a way to defer sending values to the subscriber until after the disposable is returned if they want to do so in response to a subscription. Sending them immediately in the `didSubscribe` block causes #94 regardless of how we fix it in the built in signals, and `+createSignal:` lets the user set the `didSubscribe` block directly.

`+createSignal:` should set the `didSubscribe` block to a block that returns a compound disposable, then defers calling the user provided subscription callback, then adds the user returned disposable to the compound disposable or something like that.

Getting a bit carried away, this is probably a different issue.
What about `thanks` in addition to `thx`? Also, some of the emoji can be construed as thanking, like :+1: and :metal:
I see no logical problem. JDatabase::insertid will be called only by JDatabase::insertObject and, by demand, when some extension will need. JDatabase::updateObject doesnt call it.
Interesting change, could be better implemented without same code replication, maybe doing what you've proposed for database.php
I guess it is because kpv only remembers last read page for pdf files. But for pdfreflow, each original page is typeset into multiple "virtual pages", so in order to remember the last reading position, we need to remember the offset with the last read page, so we can jump to the right "virtual page" according the offset
Why did you create this variable? It doesn’t appear to be used anywhere.
I'm just pointing it out in the spec, but the specs are irrelevant. I'm saying I'm against having `#wma` be an alias to `#weighted_moving_average`.
Yeah. I think I'll change it to be "kind", and just do a dict lookup (storing _aspect_kind as private).
Either way, I think that it should stay in the format it was in before removing pointns (which was COO).
makes sense. But you should also add an else and throw an exception for the silly cases ;-)
fid.read() wants things in terms of bytes (which tag.size automatically uses) whereas count wants it in terms of the number of elements. Thus the original one wanted tag.size/2 elements because each element is two bytes, whereas fid.read() should need tag.size number of bytes. Make sense? It should be consistently coded that way throughout the code, if not, then let me know...
please the code pushed should keep using http://jobs.ps/rss.xml the local server is just for testing because of the limitations we're experiencing due to internal network restrictions.
Same here. No need for reflection...