@Hywan I know, but calling `exit()` or `die()` outside of a class method in a library is not good practice, anyway. I see if I can come up with a better solution as a pull request, which not so invasive and has the exact functionality. I just hoped that you already had an idea.


Thanks! :D


Se eliminan si son False


Looks like "canola" was misspelled.


Esto utiliza directamente `self.env[]` puesto que sólo se utiliza esta vez, y te ahorras una línea.


I can make it immutable by cloning within the getter, will fix soon.


I still think this should be `isinstance(item, (tuple, list))` - Python doesn't make great distinction between tuples and lists and the result might easily change tomorrow to be a list.


管道的机制很有意思，也许我们该写一个类叫做 `Pipeline`，有 `in chan`, `out chan`, `err chan`， 然后把不同运行部分串接起来。


What happened to the rest of this text?


http://whatsmyudid.com

I need to create a custom reset build if data is corrupted in iCloud.

—
Sent from Mailbox

On Mon, Jan 19, 2015 at 6:52 PM, dthurman600 <notifications@github.com>
wrote:

> So what do I need to do to get this fixed?
>> On Jan 19, 2015, at 9:54 AM, Stefan Kendall <notifications@github.com> wrote:
>>
>> http://whatsmyudid.com/
>>
>> I'll get you a build that will force the app open.
>>
>> On Sun, Jan 18, 2015 at 8:59 PM, dthurman600 <notifications@github.com>
>> wrote:
>>
>> > I upgraded Big Lifts 2 the app now crashes and does not open. I deleted an
>> > reinstalled it with no luck.
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <https://github.com/stefankendall/Big-Lifts-2/commit/b90f4c33b9787cacd598474dee265145c575e49f#commitcomment-9333200>
>> > .
>> >
>> —
>> Reply to this email directly or view it on GitHub.
>>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/stefankendall/Big-Lifts-2/commit/b90f4c33b9787cacd598474dee265145c575e49f#commitcomment-9348393


I also need to state the I'm not gonna use the CM Hudson supported devices check system, I prefer to use my own locally.


It's freed at line 381. Or what do you mean?


fixed in #9


Nitpick: perhaps change `path` to `source_path` to indicate what is copied where? (I know, copying out of dump dir does not really make sense ...)


Yes, it was added for D [here](https://github.com/fishman/ctags/pull/133/files#diff-9c07e5e602aac928d6d779592925a132R2831).


Maybe rename to `self.callback_url` to indicate that this is a `str`, not a callable.


Duh, this slipped in from earlier experiments with an `--examples-dir` option. I'll remove it.


i guess it's just my personal preference of "more, smaller files" rather than "fewer, larger files"


Will afterAsterisk be


Not sure how to explain in formal grammatical terms since, as a native speaker, English grammar is more like instinct to me :)

"Generated files successfully" is correct because it it informing the user that multiple files (plural) have been generated.

"Error in file generation" is informing the user that there has been an error in the process of "file generation". This process can result in multiple files being generated but the singular is still used for the process itself. I guess "file generation" is effectively a compound noun?


@ejacobsohn I will add comment on the BPD and SEED.  Would you like to see links to the DOE website?


а чо так? Не можна було повністю на JavaScript згенерувати ці лінки?


Eh? Why like that and not like str(v)? Sint it essentially the same?


Maybe you can add test about button disable when draft is selected


Just to be sure, doesn't https://github.com/django/django/blob/master/django/db/models/query.py#L763-L765
apply here?


Can tag start from KDocTokens.START?


It just has to do with how the cross itself behaves in terms of em sizes.


Changes look good to me.


This script could probably use something like `if OBJECT_ID('dbo.Thumbprint', 'FN') is not null exec('drop function dbo.Thumbprint')` so that subsequent Setup actually loads new meta.


change to lowercase


This isn't a revert of the entire commit d427ca3.  You should say "partial" revert in the comment.


@cvrebert consider change to https://github.com/markdown-it/markdown-it , because core devs commit there after 1.5.1 . See histories & changelogs for details.


Nah go ahead, although it's not working for vanilla server jar though because of the way Mojang changed their launcher.


OK


Very good!


I suggest to rename commit with mentioning this move.


Log with sentry here?


You could add other kinds of whitespace like `"\n"` newline characters or non-whitespace word separators like `/`.


Otherwise looks good! No improvements to make here.


Are you sure you don't want toLocal8Bit here (and elsewhere)?


Still not working. If you lose focus from the field, the value comes back.


I'm kind of attached to the concept of reference docs using stripe's layout, with nav tabs to switch between programming languages. I'd really like to bring this back somehow...

Did you see it and try it before you yanked it?


Are you also tracking the referring website and/or browser information? I can imagine the referring websites might be helpful. :-)


Looks good. In Ruby, you can write more expressive loops with Array#each:

```ruby
list.each do |item|
  if item == item_to_count
    # ...
  end
end
```


`self.raw['path'].startswith('/)`


params are getting passed as array to controller first arg


Although this logic is right, I think handling draftCount inside hasUnreadselected some how looks weird, and Array.some will not traverse all the elements actually. Maybe we could have var both allDraft = selected.selectedList.every(...) and  var hasUnreadselected = selected.selectedList.some(...)


@mouhsinelonly What did you expect? this is the dev branch. Please check out L4.2 or wait for L5 to be at least in beta.


foo isn't used!


usages


This looks a like an ordinary SequentialCursor, e.g. what you get by calling Cursor.Create(int).


That's weird. Last time i tried on master (like 1 week ago) it worked like a charm ...


Why does OsfStorageFileMetadata` ensure a leading `/`, but `OsfstorageFolderMetadata` does not?


ColumnLayout, no?


Strange... add comment?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1095 of 1095 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


:dart: you got it.


Best commit message so far.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1095 of 1095 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


done 的类型为什么是 `chan struct{}` 而不是 `chan bool`。bool 更加符合语义


This was deleted automatically when we used the single shovel


I know that they changed the ftb launcher, and I updated on the master branch. I will take over some of your ideas though. And give you credit ;)


C++11 supports typed enums, so I think its a good idea to be explicit here.


I'm using Debian :-)
Perhaps this is some kind of "race-condition": I installed the packages earlier, bit rebooted prior to configuring the shares.
If it works for your case, we can just ignore my case.


"CloudFiles"


I though it was working for an old version of vanilla mc server but not with recent versions but i might be wrong. Are you sure it's working with 1.8.x ?
No worry, i think your tool is useful and has some potential.


Since this line is returning the _return value_ of `puts`, what is the return value of the method?


There is no need for creating the mount points, `mount` will do this:
```
= state
       [...] If specifying `mounted' and
        the mount point is not present, the mount point will be
        created. [...]
```


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


This is a good message, although isn't the mean of `[-10, 10]` = `0`?


Lol. I really only did this because we needed to push an otherwise "useless" change to the server on the deploy check. Best I could come up with ;)


Alright,  i will make that change as well then.

On Tue, Jan 20, 2015, 8:30 AM kalloc <notifications@github.com> wrote:

> Yes, it makes code more readable than without type.
>
> In go1.4/src:
> y@y src $ ag --go -R ' := ' ./ | grep -v github | grep -v test |wc -l ->
> 18272
> y@y src $ ag --go -R ' = ' ./ | grep -v github | grep -v test |wc -l ->
> 59072
>
> also you can use := for next statements:
> smth_size := len(smth)
> or
> smths := make(Smth, ...)
>
> We can understand variable's type.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pki-io/pki.io/commit/3e4bee2a7d9f8ecf7c073b7cfef2ded53bad235c#commitcomment-9359297>
> .
>


No, not really. Will fix.


Have fun with starting out programs!


not sure wtf that was doing there... - removed


Yes, that would be helpful.

From: Gavinhastings [mailto:notifications@github.com]
Sent: Tuesday, January 20, 2015 11:50 AM
To: hpxmlwg/hpxml
Cc: ejacobsohn
Subject: Re: [hpxml] Update opportunities_benefits.rst (2f6355f)

@ejacobsohn<https://github.com/ejacobsohn> I will add comment on the BPD and SEED. Would you like to see links to the DOE website?

—
Reply to this email directly or view it on GitHub<https://github.com/hpxmlwg/hpxml/commit/2f6355f7da2fc03200eaff8f54fabc5e0db6aee3#commitcomment-9359658>.


The code `(height*2)-1` seems complicated enough to deserve a name. Maybe a variable like `level_width`?


Thanks :) I was kind of about to give it up as nobody cared about it, and I was the only person actually using it.
But I ran both an ftb and vanilla server with it for about 2 years, and I never had to reboot any of them (well, the ftb server got corrupted once and I had to manually roll back, but that doesn't count)
The only thing missing for it to be comlete is a backup program. Which I wrote last week (MCBackup), but it's quick and dirty, and the code is by far not as clean as this MCDeamon.
Thanks for your appreciation and your effort!
And I like the travis thing, I'll definitely merge that to master. (And most of the other things)
I'll check again for 1.8.x.


@goodwinnk By the way, Java parser doesn't allow unclosed multiline comments. Maybe we shouldn't, either?


Can you confirm these are the correct sites for this document?

http://energy.gov/eere/buildings/building-performance-database

http://energy.gov/eere/buildings/standard-energy-efficiency-data-platform

From: ejacobsohn [mailto:notifications@github.com]
Sent: Tuesday, January 20, 2015 10:07 AM
To: hpxmlwg/hpxml
Cc: Gavin Hastings
Subject: Re: [hpxml] Update opportunities_benefits.rst (2f6355f)

Yes, that would be helpful.

From: Gavinhastings [mailto:notifications@github.com]
Sent: Tuesday, January 20, 2015 11:50 AM
To: hpxmlwg/hpxml
Cc: ejacobsohn
Subject: Re: [hpxml] Update opportunities_benefits.rst (2f6355f)

@ejacobsohn<https://github.com/ejacobsohn> I will add comment on the BPD and SEED. Would you like to see links to the DOE website?

—
Reply to this email directly or view it on GitHub<https://github.com/hpxmlwg/hpxml/commit/2f6355f7da2fc03200eaff8f54fabc5e0db6aee3#commitcomment-9359658>.

—
Reply to this email directly or view it on GitHub<https://github.com/hpxmlwg/hpxml/commit/2f6355f7da2fc03200eaff8f54fabc5e0db6aee3#commitcomment-9359976>.


My pleasure.


For real list:
* first item
* second item

And we're done


Yes I've seen it before and always liked it.

We decided to start from a fresh slate and incrementally bring back content, including all the documentation/reference so that everything is nice and clean :)

I've added an item to the todo list on the trello card: https://trello.com/c/pmu3HORy/164-move-blog-school-training-under-cukes-info


Thanks for including your notes!


test


You can write `puts` with no arguments to print a new line.


Check out [Integer#downto](http://www.ruby-doc.org/core-2.2.0/Integer.html#method-i-downto)


This is probably leaking a global here.  Don't worry about caching the length, you can just keep checking against `colorList.length` in the condition.


Will make the required Changes :)


It's not used!
Any particular reason behind committing it ?
If it's not necessary, please remove it.


Sorry, this isn't leaking a global... Although we still don't need to worry about caching that value


For real list:
* first item
* second item
And done


Please try to avoid committing whitespace. You can customize your text editor to remove whitespace at end of line every time you save. Just Google that last sentence and replace "text editor" with whatever you use.


Was this tested on ubuntu/debian? If not, i'd leave those classes off until someone can test.


Yes, those appear to be the correct links.

From: Gavinhastings [mailto:notifications@github.com]
Sent: Tuesday, January 20, 2015 12:09 PM
To: hpxmlwg/hpxml
Cc: ejacobsohn
Subject: Re: [hpxml] Update opportunities_benefits.rst (2f6355f)

Can you confirm these are the correct sites for this document?

http://energy.gov/eere/buildings/building-performance-database

http://energy.gov/eere/buildings/standard-energy-efficiency-data-platform

From: ejacobsohn [mailto:notifications@github.com]
Sent: Tuesday, January 20, 2015 10:07 AM
To: hpxmlwg/hpxml
Cc: Gavin Hastings
Subject: Re: [hpxml] Update opportunities_benefits.rst (2f6355f)

Yes, that would be helpful.

From: Gavinhastings [mailto:notifications@github.com]
Sent: Tuesday, January 20, 2015 11:50 AM
To: hpxmlwg/hpxml
Cc: ejacobsohn
Subject: Re: [hpxml] Update opportunities_benefits.rst (2f6355f)

@ejacobsohn<https://github.com/ejacobsohn> I will add comment on the BPD and SEED. Would you like to see links to the DOE website?

—
Reply to this email directly or view it on GitHub<https://github.com/hpxmlwg/hpxml/commit/2f6355f7da2fc03200eaff8f54fabc5e0db6aee3#commitcomment-9359658>.

—
Reply to this email directly or view it on GitHub<https://github.com/hpxmlwg/hpxml/commit/2f6355f7da2fc03200eaff8f54fabc5e0db6aee3#commitcomment-9359976>.

—
Reply to this email directly or view it on GitHub<https://github.com/hpxmlwg/hpxml/commit/2f6355f7da2fc03200eaff8f54fabc5e0db6aee3#commitcomment-9360017>.


And again:

* item one
* item two


So where _would_ we go?


Thanks ncouture, fix applied and looks to be working.


1.5 is full of those, I just wanted to follow what is already there! Maybe undefine FALSE and TRUE in a globally included file to force people use false and true


Sorry, I was wrong, however, I already made a [pull request](https://github.com/AtomLinter/linter-clang/pull/22) including this and some more.


Nice work, @theodoretripp! Thanks for including your comments and thoughts along the way.

In the future, if you want feedback on multiple files, it's a good idea to make separate commits and then combine them all into a pull request.


Ideally we don't need to do this ^^ because for symbolizers like markers (that are a very heavy compile) I worry that this will 4x compile times.


It's working @QsBBQ !


I can't understand this addition...


I will get with Maggie at APS today and see if we can pull that from her.


Nice use of the default! Avoided having to write extra logic later. :star:


Use d2-common and your particular variant's tree (d2vzw, d2att, d2spr, etc)


Wow this looks really good, thank you for this. Mind opening a PR?


Got it.

I will change the code per your comments above.

BTW, do you like another git commit so that you can review incremental change, or do you want me merge 1st and 2nd commit for new pull request?

Maybe the commit amend should be done after code review is done.

I don't have lots of github dev experiences. Please let me know the best practices here.


Вже переробив


You bet.


Duh. #shameonme :P


From https://github.com/styleguide/ruby =>

> Prefer {...} over do...end for single-line blocks. Avoid using {...} for multi-line blocks (multiline chaining is always ugly). Always use do...end for "control flow" and "method definitions" (e.g. in Rakefiles and certain DSLs). Avoid do...end when chaining.


Looks great!


A ne bi bilo lažje popraviti .txt? :)


This is great! Can test on Win 8 if you guys would like but I think the deal is the same


Oops, let me put it back although it's not doing anything.


See 9d3685d


The fix looks good. I need to test it.


Ack


Yes. There are two features of Ruby that you are using here:

1. The **implicit return** (the value of the last line of a method is the return value for that method)
1. Variable assignment is an **expression** which evaluates to variable's assigned value

The fact that variable assignment is an expression in Ruby lets you write code like this:

```shell-session
$ irb
irb(main):001:0> foo = (bar = 1)
=> 1
irb(main):002:0> foo
=> 1
irb(main):003:0> bar
=> 1
irb(main):004:0>
```


GG :+1:


Yep.  Thank you.  You were the key in solving this.  It looks sooo simple but omg this is just the exact right permutation of serveral similar things.


It was because older browsers don't understand the keyword "currentColor".

I've put some fixes in for Safari that enables hover effects in older browsers. (see https://github.com/Financial-Times/o-overlay/commit/f88953e7ff36b66410ab9466860e3e4778f24cb2#diff-9f9f294f604cd7407e7622a6d0896102R53) so that comment doesn't apply anymore.


Usually, though, you won't see any variable assignments on the last line of a method. Might as well just return the value itself. ;D


The `oOverlayCompactCloseIcon` (compact) one was.


Good question! And nice code.

One case that you're not accounting for, however, is if you try to find the mean of an empty array. What happens then?


If you get a chance to run a few tests that would be great.

I tested the following on my machine:
1. Run the script where nothing (except for git) is installed.
2. Run the script with a few of the dependencies installed.
3. Run the script with all dependencies installed.


See PR #1432


See PR


Yes, there is a typo.

Harmless - it just meant that the title was always sent in data to browser when it didn't need to be.


Looks good!


[x] done


@josephnoir please open an issue for this. Having this as a comment in a commit somewhere almost guarantees it will be lost.


## This is a markdown file


deleting


`string` isn't a type.


added new c4 instances as suggested from this pull requrest https://github.com/jtriley/StarCluster/pull/488


The only problem was that I had included the debug .exe and not the release version. The code changes doesn't matter.


Please fix the incorrect indentation introduced with this commit, otherwise the plugin doesn't start anymore


https://github.com/DeveloperLiberationFront/socaster-api/blob/df52896cbc1d12813f09afac2678577031c77e9b/run.py#L118

is what the commit message is referring to.  I'm seeing
```
E 150120 12:06:56 http1connection:53] Uncaught exception
    Traceback (most recent call last):
      File "/usr/local/lib/python2.7/site-packages/tornado/http1connection.py", line 234, in _read_message
        delegate.finish()
      File "/usr/local/lib/python2.7/site-packages/tornado/httpserver.py", line 280, in finish
        self.server.request_callback(self.request)
      File "/usr/local/lib/python2.7/site-packages/tornado/wsgi.py", line 280, in __call__
        WSGIContainer.environ(request), start_response)
      File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1836, in __call__
        return self.wsgi_app(environ, start_response)
      File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1820, in wsgi_app
        response = self.make_response(self.handle_exception(e))
      File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1403, in handle_exception
        reraise(exc_type, exc_value, tb)
      File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1817, in wsgi_app
        response = self.full_dispatch_request()
      File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1477, in full_dispatch_request
        rv = self.handle_user_exception(e)
      File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1381, in handle_user_exception
        reraise(exc_type, exc_value, tb)
      File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request
        rv = self.dispatch_request()
      File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request
        return self.view_functions[rule.endpoint](**req.view_args)
      File "/usr/local/lib/python2.7/site-packages/eve/endpoints.py", line 55, in collections_endpoint
        response = post(resource)
      File "/usr/local/lib/python2.7/site-packages/eve/methods/common.py", line 229, in rate_limited
        return f(*args, **kwargs)
      File "/usr/local/lib/python2.7/site-packages/eve/auth.py", line 57, in decorated
        return f(*args, **kwargs)
      File "/usr/local/lib/python2.7/site-packages/eve/methods/common.py", line 675, in decorated
        r = f(resource, **combined_args)
      File "/usr/local/lib/python2.7/site-packages/eve/methods/post.py", line 188, in post
        getattr(app, "on_insert_%s" % resource)(documents)
      File "/usr/local/lib/python2.7/site-packages/events/events.py", line 77, in __call__
        f(*a, **kw)
      File "/usr/local/src/socaster-api/run.py", line 118, in insert_events
        db.tools.insert({application:item['application'], name:item['tool'],users:[]})
    NameError: global name 'application' is not defined
```


Maybe we should make debug an option as well.


[Medium] This shouldn't be part of *AbstractProduct*. If you want to include 2 products, it should be in the Offer. You can think when you go to the supermarket. There are 2 options when you have the same product twice (or even more)
- They are added to your ShoppingCart (or ticket) individually.
- They add 1 product and they set the number of same item.

So, if you want to modelate this same behaviour, you don't have *units* attributes, but you can add a method in your shopping cart to do something like:
```java
  shoppingCart.addProducts(product, number);
```
And internally just include the *product* *number* times in your list.


WebRTC


"at de store planene"
"jevn jobbing hele semesteret"
"forrige eksamensperiode"
:lollipop:


I'm leveraging Symfony's `Symfony\Bridge\Monolog\Handler\SwiftMailerHandler` which ships with the Monolog Bridge. So I'd end up extending SwiftMailerHandler above and end up coupling my implementation to all the handling logic. I'm not so keen on that.

Creating the template Swift_Message feels like a different responsibility than handling the log levels + coordinating the dispatch of email, which is the responsibility of SwiftMailerHandler. The current SwiftMailerHandler implementation already supports keeping its responsibilities separate from creating Swift_Mailer so extending it to customize that behavior feels wrong.


"hjelp til" :lollipop:


Why did you remove `import ctypes` -- we use `ctypes.DLL` below


Thanks, Nick. That looks great. You really filled it out better than I
pictured.

John

On 01/20/2015 04:56 AM, NickLambert wrote:
>
> Apologies for the delay in getting to this John. I've incorporated
> your very valid point and have incorporated this into the new
> definition. Hopefully the explanation is now more accurate, however,
> if you still feel it falls short please feel free to suggest
> improvements.
> http://systemdocs.maidsafe.net/content/how_to_use_it/an_end_user.html
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/maidsafe/SystemDocs/commit/5352d98e638ba59fe26c8b60d51dd171501d0cff#commitcomment-9353814>.
>


Why did you change this back? Now it's wrong again...


[Medium] Add a proper comment


Using atomic may shut up unnecessary messages from tools like valgrind. I haven't run valgrind in a while, but would want to verify this doesn't cause it to complain about racy access.


Ack


[Medium] Use a proper name for this interface


happens to the best of us.

![](http://25.media.tumblr.com/e8de2e5ddd901613be15ddf088844fa2/tumblr_mxfh97VW751t4ar2no1_400.gif)


Looks good! Nice work. Only adjustment would be that it'd be nice to point out that the method behaves in that particular way, i.e. a comment pointing out that it will return the first longest string if there are more than one.


Det ser veldig bra ut :bouquet: :lollipop:


Looks good!


Please either a comment here that `has_gss_framework` is a Mac-specific thing, or rename it to `osx_has_gss_framework` or something like that -- it makes it easier to understand what the variable means without having to look back to its definition.


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=d0d3a23d18be4fd49c0435cef7878bbb
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [SUCCESS]


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (597 of 597 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-2.36%25-green.svg)


Adding whitespace here. A suggestion is to add some sort of icon for tailing whitespace (I use . character for tailing whitespace):

![image](https://cloud.githubusercontent.com/assets/526122/5822344/62368884-a0ba-11e4-9186-d271d3430ad1.png)


The subsequent commit added `allow_None` to `Parametered`, so I think `allow_None` should be removed from here & `__init__`


[x] done


Actually this doesn't fix anything but it's the correct start and it maybe makes kexec load slightly faster.
This is also correct for shinano.


It is possbile, with some special injection stuff.
See https://github.com/SpongePowered/Sponge/pull/140 for how I did it in Sponge.


I need to rewrite this role and split it into two, client and server. I'll take this into account, thanks.


I think we should leave this, because it may be something people forget to do.


Thanks! For future reference, you should avoid committing unused, commented-out lines like this. But thanks for explaining!


Whatever... as long as it is merged in the next release :)


Nice work! No room for improvement here.


Hey, I wrote a [lengthy post](http://stackoverflow.com/questions/21475001/using-the-doctrineobjectconstructor-how-how-are-new-entites-intended-to-be-crea) asking about this issue, and this repo was so I could have that discussion.  I never got an answer, so I'm still not sure if not creating the object is intentional, or a bug.

Since it appears that at least one other person agrees with me on this, I'll make a pull request.


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3302/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3288/


(I'll do this, along with adding `allow_None` handling to `List` and anywhere else necessary.)


Fixed:
https://github.com/qcad/qcad/pull/7

On Mon, Jan 19, 2015 at 1:57 AM, Andrew Mustun <notifications@github.com>
wrote:

> We have received a bug report about this contribution:
> http://www.qcad.org/bugtracker/index.php?do=details&task_id=1166
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/schmidtw/qcad/commit/ba4b2627be44ed6216965afd47bada9fda01e17a#commitcomment-9336485>
> .
>


No entiendo por qué no sale el autor si me pidió identificarme como usuario en github


This should be an enum class to eliminate the possibility of accidental implicit conversions with int.


@rodrigopavezi I believe it should be either "it will intercept all the requests, get specific information from them then record a log entry with the extracted information." or "it will intercept every request, get specific information from it then record a log entry with the extracted information."


This is still racy.

PUTFH may have gotten a fsal_pnfs_ds reference and cached pds->serving, but not yet taken the export reference.

The export reference taken in association with the DS needs to be taken inside pnfs_ds_get while holding the server_by_id.lock. THEN we can rely on the idempotent removal.

And we may not need pds->draining.

And pds->serving SHOULD only be accessed by pnfs_ds_get, pnfs_ds_remove, and the FSAL code that sets it in the first place.


I think there is still one test failure that is related to that change:

http://jenkins.plone.org/job/plone-4.3-python-2.6/2625/console

No idea why this test fails on Python 2.6 and passes on Python 2.7.


Im extremely confused on how d2-common works. How do I use it in
combination with the specific tree?

On Tue, Jan 20, 2015 at 11:18 AM, Dan Pasanen <notifications@github.com>
wrote:

> Use d2-common and your particular variant's tree (d2vzw, d2att, d2spr, etc)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CyanogenMod/android_device_samsung_d2lte/commit/e9ba3d5766930619356231877a67174f7c1772bb#commitcomment-9360186>
> .
>


Un comentario


Thanks.  Fixed.


Good call.  Fixed in https://github.com/cantino/huginn/commit/035097bf769724626e9806f94f6f4efeaa0d5296


You could check out [String#center](http://www.ruby-doc.org/core-2.2.0/String.html#method-i-center). Otherwise the only suggestion I have is to build up your string for each line _before_ printing it, like this:

```ruby
spaces_str = spaces * " "
asterisks_str = asterisks * "*"
line = spaces_str + asterisks_str + spaces_str
puts line
```


I did but suspect it was too late. Sorry.


Added test


@goldoraf  I have few questions regarding Bosonic.Can you provide answer
1. what is the difference between bosonic-platform and bosonic-runtime file?
2. There are many code repo like bosonic,platform,runtime,bosonic.github.io !!! what are purpose of these many repos? which one is the correct repo to follow?


Don't worry... it's me but I've missed to add git profile into [SGit: Git Client for Android](https://play.google.com/store/apps/details?id=me.sheimi.sgit&hl=en)


Good idea, but lets define some of the terms and then update. ok?


What about timezones and offsets?


I found another solution to customize the behaviour...

https://github.com/schmittjoh/serializer/issues/333#issuecomment-70670368

With this you don't need a pull request, using the service alias by config is enought I think...


This is incorrect.


Absolut genial. Wir haben grad nach 5h Ma3 lernen das Game gezockt und den Boss gesehen. Hammer :D


Good point; this is different. I'll look into changing this back.


In this case, there is some implicit meaning for what `height / 2` and `height - i` represents that would be clarified by assigning them to variables before using.


А переменные точно в ```SCOPE_LOCAL``` попадают и не перезаписывают глобальные для шаблона (описание, ключевые слова и т.п.)?
После этого все локальные переменные удаляются или так и остаются висеть в глобальном пространстве имен?


Why not `time_t`?


Good catch


As of Jan 25

> You will be automatically moved to a [free subscription plan](https://mailtrap.io/pricing) and all your inboxes, except for the primary one, will be visible, but non-active.

Will this break ATH? (CC @olivergondza)


`verify=False` because there's apparently some mismatch between the domains in GitHub's certificate:

```
[pferreir:/tmp]$ openssl s_client -showcerts -connect uploads.github.com:443
CONNECTED(00000003)
depth=1 /C=US/O=DigiCert Inc/OU=www.digicert.com/CN=DigiCert SHA2 High Assurance Server CA
verify error:num=20:unable to get local issuer certificate
verify return:0
---
Certificate chain
 0 s:/C=US/ST=California/L=San Francisco/O=GitHub, Inc./CN=*.githubusercontent.com
   i:/C=US/O=DigiCert Inc/OU=www.digicert.com/CN=DigiCert SHA2 High Assurance Server CA
```


[High] This should be a *synchronized* method.
You can check this example: https://github.com/iluwatar/java-design-patterns/blob/master/singleton/src/main/java/com/iluwatar/ThreadSafeDoubleCheckLocking.java

or read the *Thread Safe Singleton* in this post: http://www.journaldev.com/1377/java-singleton-design-pattern-best-practices-with-examples.

Also read the 4 steps for a singleton here: http://www.javabeginner.com/learn-java/java-singleton-design-pattern. It's important to override *clone* method for singletons


Looks good to me.  Thanks for fixing this -- it was on my list of stuff to fix but appreciate you getting it first.


publish this site


why drop the square brackets btw?


I took a similar route when I ran into this.  I declared the fixed version you see above as a service in Symfony, then aliased jms_serializer.object_constructor.  Example:

```
<!-- Make JMS Serializer use an alternate object constructor that works with Doctrine. -->
        <service id="mystuff.doctrine_object_constructor" class="MyStuff\CommonBundle\Service\DoctrineObjectConstructor" public="false">
            <argument type="service" id="doctrine" />
            <argument type="service" id="jms_serializer.unserialize_object_constructor" />
        </service>
        <service id="jms_serializer.object_constructor" alias="mystuff.doctrine_object_constructor" public="false" />
        <!-- Done. -->
```

It hasn't bitten me yet.


I committed this from one of the shared computers--didn't want to leave my
email in the git global settings, which is where that contact thing comes
from. You guys know how to get in touch with me :)

On Mon, Jan 19, 2015 at 8:27 PM, Ian <notifications@github.com> wrote:

> Ben your email is setup wrong , you might want to double check that
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FRC2605/CodeDevelopment/commit/f97d6ce405a820f13bdb620cb7ca6696aad7291a#commitcomment-9349980>
> .
>


Also it allows to process strings larger than Word64 chars by md5String function correct.


Note to self: I'm not sure I understand the purpose of this data structure. Ask tomorrow and/or drink more coffee.


Also fixed a typo....


Shall the function comment not appear _above_ the definition?


@mried, I've added the skip check back (by moving the event emission to the factory) in 293e251. Does that look right to you?


Thanks! Is there any shot at us getting a test per the [contributions standards](https://github.com/vmware/pyvmomi/wiki/Contributions)? This is a really *great* catch and I'm so glad to get a pull request on this topic.


`vBlock` is `block` field from `bemjson`
`ctx.block` is actually the same for all cases except when we change it inside `apply()`
`mixItem.block` is self-descriptive


This RP was taken from https://github.com/appcelerator/titanium_modules/pull/110


missing `{`


You seem to be missing the "doc" subdirectory in the URL. I get a 404 error page on it but found it here https://github.com/apitrace/apitrace/blob/master/docs/BUGS.markdown#tracing


I will look into it over the weekend


##This is a markdown file


Do we need this here ?


Of course the command was just `sbt clean compile`


Version bumped and new artifact push to resolve #1


It's true, but @sfan5 said that all compilers will optimize it away and this way `DIR_DELIM` can be an std::string or similar.


Not in python:
https://www.python.org/dev/peps/pep-0257/

This will make the documentation comment also available as SpaceConsistency.run_bear.__doc__ which allows us to show those descriptions as a help text to the user when we ask him for the UseSpaces setting.


You fixed this quick.  I was about to make an issue about it.  :+1:


And that's why TDD exists. :tongue:


Adding it back in, Wasn't sure if the example was overly long and/or overly silly. This was the original
"And so we say goodbye to our beloved pet, Nibbler, who's gone to a place where I, too, hope one day to go. The toilet. No! The cat shelter's on to me. Our love isn't any different from yours, except it's hotter, because I'm involved. Actually, that's still true."


There's no `atoi` here?


Ah that's quite a good feature from python, good to know :D


@jamalex Psst, over here! Can't flag changes on the main PR for khan-exercises.js, it gets lost. Here we use jQuery's recursive extend to maintain elements of Khan.Utils.


@jamalex This was using the non-imported urlBase that we set from outside, which caused all kinds of kerfuffle.


It seems that we'll need several more thousand lines of tests before these issues will be caught automatically.


For reference : this takes 59s on MBP 2012.


@jamalex This adds the embeddedMode flag that allows us to turn off the things we don't need.


Much better now :+1:


Is this a mismatch, just unfortunate naming of variables?


Is this PHP or JavaScript (concatenation)?


I dont think we need this condition, as we depend on that class always.


:scissors: es on responses.


Buscar


I like this addition.


rax retry tests


Yes, otherwise the symbol is not exported in `libwayland_toolkit.a` (and consequently `libwayland.so`): the declaration in the header uses `VIEWS_EXPORT`, which evaluates to nothing since we're building a different target that is not `views`. Which is why the `VIEWS_EXPORT` was removed from `ui/desktop_aura/desktop_platform_screen.h`.


In general, I don't like using empty value to mean that the environment variable/flag is not set. Sometimes, empty values are intentional and not an error. Instead, you should be able to say that a particular value must be explicitly set on the command line.

I like the approach followed by https://github.com/kelseyhightower/envconfig.  It could use improvements re: (2) but I like the general approach and the interface exposed to users.


looks like the indent is broken. tabs should only ever be used on the left-hand side of any expression


[x] done


I really like the way the new specification matches our English descriptions! For example "We require zope.interface >= 3.6.0 but we can't use 3.6.3 or 3.6.4" is now spelled `"zope.interface >= 3.6.0, != 3.6.3, != 3.6.4"`.

One detail: we don't have an English justification for why we require Nevow <= 0.10. It's cool that the new spelling made it easier for me to notice that missing doc. :-)

So, please add that doc and then +1!


Just curious: why? Isn't it enough to rely on the `infinispan-core` dependency being pulled in by the `infinispan-lucene-directory` module?


This should continue to use `context.getBean(ErrorMessageWrapper.class.getName())` instead of saying `new ErrorMessageWrapper`


@rbran100 can you explain?


first one -> done
second one -> put on my todo list


Okay, I'll buy it. +ε


@mmcc, you need another line break after the last item here.
https://github.com/videojs/video.js/blob/master/CHANGELOG.md


This line should be `ErrorWrapper errorWrapper = context.getBean(ErrorWrapper.class, ErrorMessageWrapper.class.getName());`


I have `・`. This was just a human error.


Look this up via the applicationContext like the others


Look this up via the applicationContext


com


@alexey-rise thanks a mill, oh man, I am writing too fast. I will change it.


Was keeping the inconsistency between the two sections of code consistent within each section.  The use of gThrottleData[0] is extremely rare.  When used its values are likely to result in zero anyway.  There would be no harm in fixing the section to also scale the data up and down ... but no real value either.


minor update


NOPE on so many levels.. giving the HTTPd access to executing any command as any user is a *really* bad idea.


[x] done


I'm a little curious as to this still working even in the root applicationContext. Usually by definition, if you put anything in the root applicationContext then it will apply to both servlets, or not work at all. I think that you have to segregate this bean to the REST servlet.

I'm not really sure what will happen in this case. You can test by throwing some random exception in a controller in site and see if it actually gets into this exception handler.


to


hehehe :+1:


fill


We do not use forwarding nor team inbox sharing. The whole suite sends <5 emails if I am not mistaken. So history of 50 messages should be enough, same as 2 mails/second.

But I have always seen mailtrap.io as a possible breaking point in ATH so I do not feel strongly against replacing it with something else. Do you see any alternatives?


Doesn't have to be in SWAGGER_SETTINGS, I just want to make it accessible to the rest of the code, and since this version is only for swagger docs I thought that was a good place. What kind of issues do you see with this approach?


Did someone test with at least single DPU, that it is working/brings anything?


That Looks wrong to me. "'" is only needed at end of SQL


Unfortunate naming of variables. I didn't want to change the JSON convention for fear of breaking code that expects the username to actually be the canonical name of the user.

When using LDAP, the 'username' field of the JSON reply is actually the canonical name of the account. If the account is taken from an auth_json file, however, then it is identical to the actual user name.

Sending back the accepted user name (in addition to the canonical name) would be useful when a service call with mangled credentials (e.g., CAlin.iorgulescu instead of calin.iorgulescu) would successfully pass authentication. The client would receive back 'calin.iorgulescu' as the proper name.


I made the ONLY spawn in vanilla forests. I don't know exactly what BOP has in terms of forests, but I felt like the new trees spawning in birch forests and taiga forests was weird.


``_auto_deps.py`` already has this comment:
```
        # * We don't want Nevow 0.11 because that requires Twisted >= 13.0
        #   which conflicts with the Twisted requirement above.
        #   <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/2291>
```


@xaph nasıl ama süper kod dimi?


@lawlove Thanks. Fixed with fe91ec3af5396edab45b15e546e21613785724b5


It looks fine but I've not found where you actually use it? Do we need it?


root.updateValue(wheel.angleDelta.y >= 0 ? 1 : -1)


P.S. we don't want to add Infinispan to the end user deployment. The user should be prevented to use Infinispan directly.


Never knew you could do this! Very nice, cheers for the PR :+1:


Compat: We used to throw an exception on invalid conversion, now we don't.


:dizzy_face: Hopefully this is the last side affect of the excitement around pulling in #1793. I'll fix this next time we touch the changelog so we don't have to do another force push.


If it is pathed, you don't need to manually search the path - you can just try and run it. You can simplify the block and the cut/sed by just doing

```
SDL2_LOCATION=`sdl2-config --prefix`
```


Can you please add comments, at least to public classes/methods? Before any merge to develop, please obey proper coding guidelines, see: https://team.eea.sk/wiki/display/OD/Development+Guidelines


these two lines do the same thing ...


yeah, same thing ... you are repeating code in a if/else. looks weird


done by 43779bf0414b4253babb1a9b9dae09287dcc983b


Fixed.


not clear


@stefanpenner , I must say: best commit message ever.


This should probably be `config.hpp`.


I was searching for instantinfrastructure and came across this, how are things?

I'm working on getting ii up on an Intel Edison as a usb keyboard (to choose boot device via bios/efi,), disk (dynamic grub config, maybe pxe), and network (ipxe supports usb ndis).


Excellent, thanks Robb


it might be OK but you are doing some hex parsing regardless.


Ok, found it now. Eclipse failed to refresh ;)


There is already a global [`-a` flag](https://github.com/flynn/flynn/blob/bd2ac6ed88ad4ef4192da84ecde1545e8a5fab0b/cli/main.go#L93-L103), so `--app` should be redundant.


thanks i'll have a look at that and add my grain of salt to it :)


```// Correct cursor position when sign changes.```

You don't have the problem in the other direction?


Nice found ;) I will fix it


YOU ARE AWESOME!!!! I actually don't have the first one in colors.xml but I have the other 2. I just defined it a black for now because my normal material_grey blah blah blah was causing an error but I'll work with it later. Build running to test. Thank you so much, I was going crazy. The material just calls for so many things now


Eric explained that this PR was reverted because It broke the UI. We'll hold off on this until further update.


thanks :)


I uncovered this situation by re-creating the problem a few times, and generating a stack trace of all goroutines.  one of the goroutines looks like this (while holding a mutex also):

(this was using sha 9f2794, but I was also able to replicate the problem with sha 150b7f2)
goroutine 8 [chan send (nil chan), 3 minutes]:
github.com/streadway/amqp.func·001()
        /MY_GOPATH/src/github.com/streadway/amqp/channel.go:104 +0x198
sync.(*Once).Do(0xc208078280, 0xc2464ead08)
        /usr/local/go/src/sync/once.go:44 +0xd4
github.com/streadway/amqp.(*Channel).shutdown(0xc208078280, 0xc22eea4000)
        /MY_GOPATH/src/github.com/streadway/amqp/channel.go:154 +0x80
github.com/streadway/amqp.func·002()
        /MY_GOPATH/src/github.com/streadway/amqp/connection.go:347 +0x2bc
sync.(*Once).Do(0xc208078000, 0xc2464eaec8)
        /usr/local/go/src/sync/once.go:44 +0xd4
github.com/streadway/amqp.(*Connection).shutdown(0xc208078000, 0xc22eea4000)
        /MY_GOPATH/src/github.com/streadway/amqp/connection.go:365 +0x80
github.com/streadway/amqp.(*Connection).reader(0xc208078000, 0x4d2e00, 0xc208038030)
        /MY_GOPATH/src/github.com/streadway/amqp/connection.go:458 +0x25a
created by github.com/streadway/amqp.Open
        /MY_GOPATH/src/github.com/streadway/amqp/connection.go:214 +0x3e2


Nice to see this added to NC+, but @Iceee would like this renamed to "The Restricted Section" :3


Not a biggie but can you change the IP addresses for these? Make em 255.255.255.255


You can change it for `@include oColorsFor(o-overlay-close-shaded)`


Does this make sense? Temp files are randomly named. What would this do?


We used CPLEX 12.5

On Mon, Jan 19, 2015 at 3:53 AM, merlinzone <notifications@github.com>
wrote:

> Hi thouis, can your tell which version of IBM ILOG CPLEX Optimization
> Studio you use for rhoana? in the website
> http://www-01.ibm.com/software/websphere/products/optimization/cplex-studio-preview-edition/,
> I tried the Preview edition but problem size limited, you use Academic
> Initiative version or the full version in the site https://www-112.ibm.com/software/howtobuy/buyingtools/paexpress/Express?P0=E1&part_number=D0CV0LL,D0LF1LL,D0TKXLL,D0TLFLL,D0CV2LL,D12N0LL,D120VLL&catalogLocale=en_US&Locale=en_US&country=USA&PT=jsp&CC=USA&VP=&TACTICS=&am
> p;S_TACT =&S_CMP=&brand=SSVLBB
> <https://www-112.ibm.com/software/howtobuy/buyingtools/paexpress/Express?P0=E1&part_number=D0CV0LL,D0LF1LL,D0TKXLL,D0TLFLL,D0CV2LL,D12N0LL,D120VLL&catalogLocale=en_US&Locale=en_US&country=USA&PT=jsp&CC=USA&VP=&TACTICS=&S_TACT=&S_CMP=&brand=SSVLBB>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Rhoana/rhoana/commit/5ec4545eae0beb904910cfcc926428a9409896e6#commitcomment-9335561>
> .
>


return answer


/cc @MrCurtis


This is broken. It will never load the same graph.


I think these two should be called in the opposite order


Just sync them both. If you use breakfast, it'll bring both repos in.


Can we move this calculation to else of if condition below.


Das dec Attribut ist privat und kann "von außen" so nicht verwendet werden.
decorationpart ($ARG1) {1}


argh, pushed the wrong file! In basic enum it shows an alternate method, but this pull request is really for the refactoring in nested hashes


Wow, superfast! Thanks!

---
Desde el móvil - From my phone

On 20/01/2015, at 19:27, Bruno Farache <notifications@github.com> wrote:

done by 43779bf
<https://github.com/liferay/liferay-mobile-sdk/commit/43779bf0414b4253babb1a9b9dae09287dcc983b>

—
Reply to this email directly or view it on GitHub
<https://github.com/liferay/liferay-mobile-sdk/commit/0167ab5e66d44cd3c2d324156a39ea53c4fe8f5a#commitcomment-9361375>
.


For my projects I create a module log.js that looks like this:

```javascript
var bunyan = require('bunyan');

module.exports = bunyan.createLogger({
  name: 'telegram',
  src: true
});
```

And whenever I need it I do:

```javascript
var log = require('./log');
```

Having the src flag enabled will print the source file that generated the log and I can immediately go there and check the code.

Your solution works as well. Maybe instead of hardcoding the name of the module you want to use the __filename global variable.

http://nodejs.org/api/globals.html#globals_filename


Do we really need this?


I'm confused. In my testing on 2012 r2, this approach results in the RazorBroker service being created in the WinPE context and the broker script being written to X: drive. But neither the service nor script called by the service are persisted to the system that we have just installed on disk. How is that supposed to happen?


Compression details:

Losslessly compressing http://sandy.co.in/silverpush/img/misc-img/silverpush.png could save 87.3KiB (95% reduction).
Losslessly compressing http://sandy.co.in/.../nestle.png could save 51.7KiB (95% reduction).
Losslessly compressing http://sandy.co.in/.../google.png could save 51.2KiB (91% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/home-img/clients/cn.png could save 3.9KiB (67% reduction).
Losslessly compressing http://sandy.co.in/.../audiobeacon-icon.png could save 3.8KiB (30% reduction).
Losslessly compressing http://sandy.co.in/.../kellog.png could save 3.5KiB (39% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/home-img/clients/mcd.png could save 2.6KiB (36% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/bg/product.jpg could save 2.5KiB (7% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/home-img/clients/rbs.png could save 2.1KiB (19% reduction).
Losslessly compressing http://sandy.co.in/.../verisign.png could save 1.9KiB (12% reduction).
Losslessly compressing http://sandy.co.in/.../disney.png could save 1.3KiB (11% reduction).

Losslessly compressing http://sandy.co.in/silverpush/img/career-img/skills/s1.png could save 71.8KiB (95% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/career-img/skills/s2.png could save 72.4KiB (96% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/career-img/skills/s3.png could save 79.9KiB (96% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/career-img/skills/s4.png could save 78.3KiB (96% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/career-img/skills/s5.png could save 60.3KiB (96% reduction).

Losslessly compressing http://sandy.co.in/.../targeting-icon-inpage1.png could save 6.1KiB (33% reduction).
Losslessly compressing http://sandy.co.in/.../targeting-icon-inpage2.png could save 4.2KiB (32% reduction).
Losslessly compressing http://sandy.co.in/.../targeting-icon-inpage3.png could save 2.2KiB (22% reduction).

Losslessly compressing http://sandy.co.in/silverpush/img/team-img/anuj_prakash.jpg could save 14.6KiB (9% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/adarsh_sasi.jpg could save 13.4KiB (13% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/aman_nagpal.jpg could save 13.4KiB (37% reduction).
Losslessly compressing http://sandy.co.in/.../tanvi_thakral.jpg could save 13.2KiB (50% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/mudit_seth.jpg could save 13.2KiB (32% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/prena_reddy.jpg could save 13.1KiB (38% reduction).
Losslessly compressing http://sandy.co.in/.../nishant_srivastava.jpg could save 13.1KiB (40% reduction).
Losslessly compressing http://sandy.co.in/.../vikram_tiwari.jpg could save 13.1KiB (45% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/piyush_bhatt.jpg could save 13.1KiB (50% reduction).
Losslessly compressing http://sandy.co.in/.../shipra_mishra.jpg could save 13.0KiB (34% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/ankit_paul.jpg could save 13.0KiB (42% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/rahul_gupta.jpg could save 13.0KiB (39% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/rahul_joshi.jpg could save 12.8KiB (39% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/jules_levi.jpg could save 12.8KiB (52% reduction).
Losslessly compressing http://sandy.co.in/.../pragyank_bana.jpg could save 12.7KiB (14% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/matt_casey.jpg could save 12.7KiB (28% reduction).
Losslessly compressing http://sandy.co.in/.../abhimanyu_yadav.jpg could save 12.5KiB (39% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/alex_modon.jpg could save 12.5KiB (35% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/anil_kundu.jpg could save 12.5KiB (28% reduction).
Losslessly compressing http://sandy.co.in/.../hitesh_chawla.jpg could save 12.5KiB (28% reduction).
Losslessly compressing http://sandy.co.in/silverpush/img/team-img/rimakshi.jpg could save 12.5KiB (33% reduction).


using System;
using System.Globalization; //included for dateTime objects
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace base64_conversion
{
    public partial class Form2 : Form
    {
        public Form2()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            String date1 = textBox1.Text;
            String date2 = textBox2.Text;
            String date3 = textBox3.Text;

            String format1="dd/MM/yy";
            String format2 = "dd/MMM/yyyy";
            String format3 = "MMM-dd-yyyy";

            textBox1.Text = DateParser(date1 ,format1 );
            textBox2.Text = DateParser(date2 ,format2 );
            textBox3.Text = DateParser(date3, format3);

        }

        public String  DateParser(String date1, String format1)
        {
            DateTime dtArg=new DateTime ();
            if (DateTime.TryParseExact(date1, format1, new CultureInfo("en-US"), DateTimeStyles.None, out dtArg))
                return dtArg.ToLongDateString();
            else
                return "Please enter the date value in specified format";

        }
    }
}


@shamansir here you go!


Yeah, no rush.

On Tue, Jan 20, 2015 at 10:25 AM, Matthew McClure <notifications@github.com>
wrote:

> [image: :dizzy_face:] Hopefully this is the last side affect of the
> excitement around pulling in #1793
> <https://github.com/videojs/video.js/pull/1793>. I'll fix this next time
> we touch the changelog so we don't have to do another force push.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/videojs/video.js/commit/e8030ea88fa62c7644ed78af3bba7eff2f6eae27#commitcomment-9361353>
> .
>


Shouldn't the pkg-config file be installed to `$(DESTDIR)/$(PREFIX)/$(LIBDIR)/pkgconfig`?


Consider replacing this with a new method: `!ignored.hasCompilationError()`


Leaving comment....


```_call_nikola_list``` is called two more times in ```check.py```.


Considering that you have the userSchema and postSchema in separate modules, you should import them here and register them on the connection. Once you have this each module that needs access to the database will import the db connection that you have here and it will have access to the models as well.

```javascript
var userSchema = require('./schemas/user');
var postSchema = require('./schemas/post');

db.model('User', userSchema);
db.model('Post', postSchema);

module.exports = db;
```

Now, let's say that I need access to the User model in one of your modules.

```javascript
var db = require('./db');
var User = db.model('User');

User.find(...)
```


missing space "key-value-pairto"


/////////// homework completed

using System;
using System.Globalization; //included for dateTime objects
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace base64_conversion
{
    public partial class Form2 : Form
    {
        public Form2()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            String date1 = textBox1.Text;
            String date2 = textBox2.Text;
            String date3 = textBox3.Text;

            String format1="dd/MM/yy";
            String format2 = "dd/MMM/yyyy";
            String format3 = "MMM-dd-yyyy";

            textBox1.Text = DateParser(date1 ,format1 );
            textBox2.Text = DateParser(date2 ,format2 );
            textBox3.Text = DateParser(date3, format3);

        }

        public String  DateParser(String date1, String format1)
        {
            DateTime dtArg=new DateTime ();
            if (DateTime.TryParseExact(date1, format1, new CultureInfo("en-US"), DateTimeStyles.None, out dtArg))
                return dtArg.ToLongDateString();
            else
                return "Please enter the date value in specified format";

        }
    }
}


Needs bacon


Just wondering, what's the point of random "I'll just remove this name from here"-commits?


Temp files receive random file names.


proba


"separated" instead of "concatenated"?


This file should be converted in a module that creates only the userSchema and exports it to be used inside the mongoDB.js module.

Maybe put in /schemas/user.js


[x] done


Joone can you change BUG=#352  ->  Fixes #352


This will go away because it's handled inside mongoDB.js.


already done?


Yes, you right, see irungentoo/filter_audio#7.

NOTE: in this commit I made a lot of errors and you have my apologize for not double checking that.


test


see https://github.com/emscripten-ports/SDL2/issues/2#issuecomment-70691718


This should be removed


Use build-in `random` utility from hubot: https://github.com/github/hubot/blob/master/docs/scripting.md#random


Do we need a space for `spoderman pls` query?


You don't really need to set it as `undefined`, it is already `undefined` if you don't define it.


Hi @h4ck3rm1k3

First, many thanks for the recent work on Salt. It's very much appreciated. :]

I'm a little concerned that you're going to spend a lot of time on this and that we may be in a position where we're not able to accept the work that you're putting in. I would feel terrible about that so I want to make sure we're on the same page here.

(As a bit of background, in case you aren't aware -- I'm one of the engineers on the core development team and I'm one of the folks responsible for reviewing pull-requests to the project.)

As you've seen above, there are numerous cases wherein exception handling has been already considered by the original author and optimized to a given use case. I simply don't feel like any kind of automated audit of these cases is going to be able to come to a sufficiently deep contextual understanding. There is just far too much potential for damage, performance regressions and spurious error reporting for anybody on the core development team to feel comfortable with this sort of approach. It's simply far too unrealistic for an automated approach to classify exceptions which are there for "good reason" and those which are not. That's not a criticism of your ability to implement a parser that walks through the code-base trying to identify exception handlers, it's just a critique of the general approach.

Just to be as clear as I can --  an automated approach for code changes like the one you're proposing will not be merged into Salt.

Of course, you're completely entitled to disagree and I welcome your opinion. I just don't want to be in a situation where we end up disagreeing on a PR after you've put in a ton of work.

We're totally on-board with taking these cases on an individual basis where we can carefully review each one and understand the potential impacts and benefits. That sort of thing would be very welcome and certainly appreciated. :] (That's not saying that we want to review an automated approach split up into smaller PRs. We need to understand the reasoning behind each one.)

If you want to chat on IRC or have a more-depth conversation, we can definitely arrange that. Thanks again for the work and interest you have in Salt. Cheers. :]


It's a random_id I needed.


neh that's not appropriate.


Great, I implemented your second suggestion as it reads a bit smoother to me.


Ha! That _is_ confusing. Reverting.


I am a nice comment


getDefaultGraphDirectory should be renamed to something that implies the random value. getTemp...


[x] done


Oh, thanks. You have some very slick stuff over at Sponge, hehe. I might ~~steal~~ get inspired by it.


+1. Rename it to getTempGraphDirectory()


Changed.


Should probably use path.join


Now, it's great that you have the idea to create a PostProvider kind of object, but there's no need create a new class because you can attach the methods directly to the postSchema and have them available in the model.

Please read here about "Instance methods" and "Statics" http://mongoosejs.com/docs/guide.html

So, your findAll function will be converted to:

```javascript
postSchema.statics.findAll = function(done) {
  this.find({}, done);
}
```

I usually use selections only when there're some heavy fields (memory wise) in the object that I want to avoid loading, but otherwise the optimization is not really worth. It's also useful when you don't want to load some sensitive data.

So, in the case of the "post" objects I don't think you really need to use selections.


Great.


Kernel was updated at the same time but apparently our gerrit server was on an extended union break. After a swift kick in the arse from management gerrit is back to working properly and has posted the kernel update as well. All is back in order.


Oil missspelling


tutututututu


Thanks, fixed in next commit.


now, you can call findAll on the Post model.
```javascript
var Post = require('./mongoDB').model('Post');
Post.findAll(...)
```


man lo parte el comment. ggwp


findById already exists in mongoose out of the box.

http://mongoosejs.com/docs/api.html#model_Model.findById


C'est insupportable, je ne vois vraiment pas pourquoi replace et replaceOutOfIndex ne fonctionnent pas!


You fixed `args.snapshot_name` at L80 but not here.


This is not correct, child must get path created from mPath not from mDirPath. Reverted in  74e2abd38cf. Unfortunately I could not reproduce the crash reported in the issue 11871 and fix that.


Great!


gros nul


Yep.  I thought this was platform specific.  Will update.  Thanks!


correu per en tomeu


correu amb **markdown**


fucking owner man


ahir ggwp


Set the default value to `0`.


#titol
**negreta** *cursiva*


I would use lodash here.

```javascript
var post = new Post(_.pick(['content', 'author', 'originalAt']);
```javascript
https://lodash.com/docs#pick

Also, the createdAt field should be populated on the server. You can't relay on what the client is sending because the user browser may be on a different timezone etc. Lot's of problems can occur, so it's better for the server to set up the timestamp.

```javascript
post.createdAt = new Date();
```javascript


I believe the correct spelling is `found` :)


You are right. The tests didn't pass in the first clone so I din't take them seriously but there was just an issue.


@frosner this is the code I just told you about. When you would run this test you will see that the stubbing does not work. Neither does the when(request).send().thenReturn(response) syntax. Instead the actual method `send()` of the package private class `Request` is called.

As you can see, I found a workaround. But I wonder why mockito cannot mock this method call. Thanks in advance, dude :yum:


I'd delete the if.

```javascript
done(err, savedPost);
```


Inline comment missspelling


If we pass value 0 to debug, won't it mess up whole code? finderjs takes debug only as ` false `, right?


hi @benrudolph


Where the download ?


I'm writing a comment


Good job, Navin


@benrudolph are you there?


The above patch has been backported to release-3.6 branch and has been sent for review. (http://review.gluster.org/#/c/9467/). It will be part of 3.6.2. Thanks for bringing it to our notice.

Thanks


Maybe just a comment or something. :-) But yes, a special case is otherwise needed. :-)


Looks good!


This function will be converted to an instane method in userSchema (schemas/user.js)

```javascript
userSchema.methods.toEmber() {
  ....
}
```

And then you'll be able to call it on user objects

```javascript
   return res.send({user: user.toEmber()});
```


i've recently been notified that opscode would prefer that we don't update the metadata.rb or the CHANGELOG.md file.  This is so they can merge multiple pull requests at once without having to change these.


Now might be a good time to start using postgres. Do you have it set up on your computer?


sql/updates/world/2015_01_12_00.sql to sql/updates/world/2015_01_12_00_world.sql


I've been thinking for a while of a good way to automate that date, with svn you could use tags, I'm not sure about git


> Do you see any alternatives?

`mock-javamail` (as used in `mailer-plugin` for example) is not suitable here, since it has to run in the same JVM. We would need something that starts a local SMTP server on a random port. [subethasmtp/Wiser](https://code.google.com/p/subethasmtp/wiki/Wiser) perhaps?


the problem is that nidmresults does not even raise an exceptions: https://github.com/incf-nidash/nidmresults/blob/44bbb40a003d27d082e54b54ddd51cbe46f67570/nidmresults/exporter.py#L202

subprocess.call() does not raise exception if shell=True


Can you do this and see if it works:

```js
debug: !!parseInt(localStorage.getItem("finderDebug")) ? true : false
```


well it worked but my text is black too haha


DONE in 8feb1d5cadb550c6405e8705bfe0375c2fb592fc


You could just cast to array, no ternary required


Wrong issue should have been close #9


Rather than using `vtkRendering` directly, `${vtkRenderingVolumeOpenGL2_KIT}` could be used. Other than that, I would probably rewrite the `copy_shaders` target to depend on custom commands for each shader individually then add those outputs to the library directly through the `add_library` call.

I wonder if CMake should add dependencies of `INTERFACE` libraries transitively (we allow `INTERFACE` targets to add sources, libraries, defines, etc., so dependencies doesn't sound out-of-bounds). @bradking?


in a customer scenario, perhaps.  this is just a script I use in dev, though


Separating the code from the main wrapper is good, but this is still substantial code, so it should probably be in, say,  src/python/foursquare/source_code_analysis/scala/scala_import_sorter_main.py. Then you can put a wrapper shell script in <root>/scripts.

Also, get rid of this shebang. Let the wrapper script run "python2.7 path/to/main/module" to use whatever the system's default python2.7 is. There's no guarantee that there is a /usr/bin/python, or that it's not broken (I'm looking at you, OSX).


That wold make sense. I can check if I find something to easily do that.


Substr() will return a string or false, casted to in this could never be -1?


This should be not override the libmei method (the should remain intact). A simple workaround it to rename it GetDrawingDur()


hhhhhh


wtf


restoring code


Asturianu


We need a MAX_CHORD_DEPTH (see MAX_BEAM_DEPTH) - in needs to be more than 1 in case we have app/rdg/ in between for example. The performance impact is probably negligible, you can set MAX_CHORD_DEPTH to -1 for now


The code in note.cpp can remain the same, though?


Could you reference the issue numbers?


Thanks for explanation. So "Warnings in files generation" should be changed to "Warnings in file generation" too?


The reason this is important is that I'm going to be doing something similar for Has/GetStemDir in the meantime as well.


LGTM.


saw approval from macintux
at https://github.com/basho/clique/commit/aa2edf26fc1fe7bd8f2deeb926b2de781627be00


merging basho/clique/feature/stderr_alert_output = aa2edf26 into borshop-integration-46-feature/stderr_alert_output


basho/clique/feature/stderr_alert_output = aa2edf26 merged ok, testing candidate = 04bab411


nom


Sorry if this is a stupid question, but why are you bumping the version if nothing else changed?


oh, thanks. i always get under the assumption that i have to do some
complicated pulling of blobs in each and every tree. whatever.... lol

On Tue, Jan 20, 2015 at 12:37 PM, Dan Pasanen <notifications@github.com>
wrote:

> Just sync them both. If you use breakfast, it'll bring both repos in.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CyanogenMod/android_device_samsung_d2lte/commit/e9ba3d5766930619356231877a67174f7c1772bb#commitcomment-9361536>
> .
>


This is good. I would leave it in LayerElement::AlignHorizontally so we have everything at the same place. OK?


@RobLoach
Merging.

Does GH pages serve the file structure properly, or do we need to mess with how metalsmith-permalinks


While you're here, could you please a re-try here in case of error? If the lock isn't removed due to any issue, for instance,  networking error, it will cause failure next time a change set is run.


Nice catch!

Note that this somehow worked in older versions of ScummVM - I've just retested it with a build from the 26th of December, and the spell was stored correctly. Quite odd.


This `table` is missing in the output of `pprint(model.test(test_set))` in [the notebook](http://nbviewer.ipython.org/github/halfak/Revision-Scoring/blob/master/ipython/Demo_LinearSVC.ipynb).


Change is to GetDrawingDur


HelloWorld.md
## This is a markdown file


To change this template, choose Tools | Templates and open the template in the editor.


Good suggestion.

To that I would add a feature that via config, all log levels (error, info, warn) can be re classified so to aid debugging at runtime.

Eg

LogLevel: Normal #default
LogLevel: warn=info, error=info

Or additionally:

LogLevel: all=info

Sent from my Windows Phone
________________________________
From: Ryan Leach<mailto:notifications@github.com>
Sent: 1/20/2015 6:18 AM
To: SpongePowered/SpongeDocs<mailto:SpongeDocs@noreply.github.com>
Subject: Re: [SpongeDocs] Simplify "Other Platforms" and "Log Files" (e8136f2)

Is there any platform-agnostic way of setting the current logging levels / configuration?

---
Reply to this email directly or view it on GitHub:
https://github.com/SpongePowered/SpongeDocs/commit/e8136f2ae3fb726e0d2315e53f9483f20860b0fd#commitcomment-9356609


"custruct" is a typo.


@mpc-hc: LAV Filters custom ffmpeg fork has been updated so we can use that for the stable if we want to have less custom LAV.


Could you please develop the comment? It's hard to understand that this takes care of the previous naming convention of the lock.


I think bootstrap alert is already added above on link 8.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1095 of 1095 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-68.89%25-green.svg)


you must set the encryption key. would be better I think. Not the and an together.


delete


using LZO over LZ4  for the kernel compression for the LG G3 device will reduce the decompress and load speed of the kernel on boot , and LG made lz4 compression for linux kernel at the first time


An easier way should be to use hostname command (with -I option) :
```
root@gluster1:~# hostname -i
192.168.222.31

root@gluster1:~# hostname -I
192.168.222.31 192.168.222.40
```

From hostname man pages :
```
       -i, --ip-address
              Display the network address(es) of the host name. Note that this works only if the host name can be resolved. Avoid using this option; use hostname --all-ip-addresses instead.

       -I, --all-ip-addresses
              Display  all  network addresses of the host. This option enumerates all configured addresses on all network interfaces. The loopback interface and IPv6 link-local addresses are omit‐
              ted. Contrary to option -i, this option does not depend on name resolution. Do not make any assumptions about the order of the output.
```


Do you actually want this to be all the tasks? Or just the tasks that belong to the user being shown?


Simple user validation.


These two variables are closable. But it works on jdk 1.7 . ES recommends to run above or 1.5. So use commons IO Utils closeQuietly to close resources


having error during build with  LZ4 for now i ll use LZO until fix it


I fell into this the other day.  You might want to check to make sure that array.size isn't 0 first so you don't get a divide by 0 error.


Are you wanting lines 4-23 to be comments?


You are very forgetful @DrChat. :full_moon_with_face:


@joebubna thank you for getting this fixed so quickly!


Hey Chilion, I just noticed this one. Cdowdy made a PR, and it got merged already. Thanks, though! :-)


@mcm69 thank you!


This is becoming one hell of a page :)


Thanks


What does this do?


@datahead8888 I'd imagine so. I'd prefer to leave release cherry-picks to Quintus, though. Since he has a better knowledge of what breaks when things happen then I do, he's going to have a better idea of what changes are release-friendly.


@bytemaster Stop reverting submodules with every commit.


From the mongo doc (http://api.mongodb.org/python/current/api/pymongo/collection.html#pymongo.collection.Collection.create_index):

expireAfterSeconds: <int> Used to create an expiring (TTL) collection. MongoDB will automatically delete documents from this collection after <int> seconds. The indexed field must be a UTC datetime or the data will not expire.

Seems like it won't work in this case. And what's purpose of that? Avoid stale locks? The first thing which comes to my mind is to add timestamp to the lock and remove the stale locks on the start if they're older than some threshold. Yet, I wouldn't do it. If the deployment involves multiple hosts and the change set takes a while, this could cause a collision. I think, the administrator should take care of stale locks.


ur mum

- Justin Moore

On Tue, Jan 20, 2015 at 1:33 PM, Gran PC <notifications@github.com> wrote:

> You are very forgetful @DrChat <https://github.com/DrChat>. [image:
> :full_moon_with_face:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DrChat/Gmod-vphysics/commit/329203bb861ac0bf06e1425dc68602a0dec7a324#commitcomment-9362430>
> .
>


ur mum


yesI saw that I commented on it on google+ and give the same fix
I think the build is calling lz4 to compress while it should call lz4c


Danke, ist geändert.


It OK


rude


@sj26 - Would it be possible to publish a new version of this gem containing this commit to RubyGems? Several people have stumbled across this issue since upgrading to RSpec 3.1 (see also: http://stackoverflow.com/questions/25563130/rails-rspec-treating-metadataexecution-result)


all tests pass:
success: http://buildbot.bos1/builders/test-clique/builds/28


We're running your stuff through our teamcity server on-prem, would it be much trouble for us to get an export of your build configs so that we could match as closely as possible within our setup?


:+1: thanks man! it was fun learning how to do this!


The nymIDSource should never be thought of as a source of signatures.

It's not correct to say that the keyCredential is signed by a nymIDSource - it's signed by a master credential.

The fact that a keyCredential and and a masterCredential share the same nymIDSource is how you know they are both part of the same nym.

The fact that the nymIDSource is the signing key of the master credential is how you know the nym is self-signed, instead of being validated via some external mechanism.


you sure you need as.character? `stri_sub` does that aytomatically


noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob


@Tag0Mag0 also, just use `array.reduce` here. More idiomatic.


@SonOfBowser That `q` looks like a typo. If you remove it, does everything else go fine?


Published to http://vui-docs-internal.s3-website-us-east-1.amazonaws.com/18_655f698696640e9ccd71fb90c9b7942233e4cfeb


isn't this the same as?
```
return (depth % 5) == 0;
```


This exception will always be triggered! Method not working.


It looks like it can be anything. 9000 is what android uses in all documentation.


if we want this to ever be portable outside the browser, we need to avoid attaching things to the window. the consumer of the lib should get to decide what textsecure and axolotl get attached to.


How do you get this log output ?


Willing to use it!


my kingdom for something.something.getGroup(groupId) rather than string concatenation all over the place.


fsync just ensures that the data is flushed to disk but it doesn't ensures that the data gets propagated to other mongodb instances in the cluster. I think, your first revision with w=99 and j=True actually made sense.


PHANZU LAURENT


LGTM


Perfect.


.erlang.cookie should be chosen / generated by you before you create the docker image.  Each container instance of the image will then share the same cookie and be able to communicate via epmd.


@Luiji - this was fixed with a separate commit afterwards.


Thanks for fixing that! Please, address my comments and I'll merge the changes.

My advice would be not to relay on the pyliquib locking but introduce your own way to select the master host which will be only one running the change sets.


@swabianeagle https://github.com/hybridgroup/rubyserial/compare/v0.2.1...v0.2.2 there were a few windows specific changes


Nopepad+


Карочи, внутри треда мы не знаем сколько страниц содержит борда. Пусть будет десять. Если на борде их на самом деле меньше - ничего страшного, загрузится пару лишних 404.
Тупо в лоб, однако тем не менее.


Never mind, didn't see the pull request just before that. I'm not a git expert as you can see.


:heart:

2015-01-20 20:40 GMT+01:00 Peter Cauty <notifications@github.com>:

> noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob
> noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob
> noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob
> noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob
> noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob
> noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob
> noob noob noob noob noob noob noob noob noob noob noob noob noob noob noob
> noob noob noob
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dreamvids/DreamVids/commit/ba997a1b95b4713d120a42598525b738628c2777#commitcomment-9362553>
> .
>


I think this might clean up the ansible provisioning then in terms of which directories need to be created, allowing us to possibly delete this step


We should not override the original (model) methods generated by libmei
because this will create a some confusion. As I mentioned it, we already
have a m_drawingStemDir that looks a different places where the stem dir
can be specified (Note or Layer) or calculate it otherwise. This could go
in a SetDrawingStemDir method and be made accessible through a
GetDrawingStemDir instead of having it public and set in DrawNote. Dur
would work the same.

On Tue, Jan 20, 2015 at 8:24 PM, Andrew Horwitz <notifications@github.com>
wrote:

> The reason this is important is that I'm going to be doing something
> similar for Has/GetStemDir in the meantime as well.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DDMAL/verovio/commit/610592fc4a831e151cf0c3cac0bacbaafcd5117d#commitcomment-9362276>
> .
>


RDEPEND qca[openssl]?


Subtle but sweet.  +1 for less parameters


Раньше было 5, кстати.


OK


Forgot to remove it?


sorry this branch i have do push i don't want.  when did push of other branch fetcher


Yep, it was fixed in e90ce958884a4acde8a8e284fab251e45e6d5859


`(int) "-1" === -1` ;-)


I was thinking about `mock-javamail` deployed via special plugin alike form-element-path. Let's see if it breaks.


`from` там точно нигде внутри замыканий не используется? Или переменная `i` была лишней?


Changed to primitive


i am working on this


Où serait le fun sinon ? :-p


This might be what is causing the alignment issue. I believe the class should be ```col-xs-12```.


Provade göra en merge för att se hur det fungerade.


Можно сделать 5. Или 6.


You could pass a LatLng object instead of gps.  This would make it so you only call `new LatLng(gps.getLatitude(), gps.getLongitude())` once instead of twice.


Thank you @AlexDovgan! Will review and merge as soon as I can.


"Unsafe to use RGB_565 on KitKat or Lollipop, ignoring setDecodeFormat" => In fact I think it is safe on Lollipop as I never noticed this bug on Lollipop. It is unclear if it is safe below KitKat.


@kerbylav Спасибо, я либо не знал либо забыл по причине не использования присваивания логических операций в переменных.


this looks like it might change the behavior of value filter exclude for empty strings. Probably doesn't matter unless someone is erroneously trying to match that, but fyi etc.


You should use the Bukkit API (not the Spigot one) to be compatible with CraftBukkit and Spigot.


> It's not correct to say that the keyCredential is signed by a nymIDSource - it's signed by a master credential.

Did I write otherwise? I wrote that the nymIDSource signs (new: authenticates) the A/E/S keys.

> The fact that a keyCredential and and a masterCredential share the same nymIDSource is how you know they are both part of the same nym.

Good, I did not realize that. Will amend.

Thanks.


```col-am-4``` should be ```col-xs-4```


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1939](http://teamcity.monogame.net/viewLog.html?buildId=5010&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Done


you can just set positive to x > 0 ie `positive: x > 0`


@dkorunic This is mistake, I guess? Personally, I prefer relative path over absolute one in this case...


these were sorted in lexicographical order, so `go fmt` might just put them back. If that's the case, this commit will be deleted.


Well, that was an anti-climactic fix. :)

Looks fine, could you commit to master? Thanks.


blabla


where is this.href coming from?


haha i can't believe i didn't see that. thanks!


Hvala, bom popravila, danes sem na vajah samo na hitro popravila, ko sem
opazila, da je narobe :)

Dne 20. januar 2015 17.36 je Janoš Vidali <notifications@github.com>
napisal/-a:

> Opozorilo za NA se zgodi zaradi črk na koncu časov v tej datoteki (4
> pojavitve). Drugod ni teh problemov. Lahko bi sicer poskrbela, da ti iz
> časov pobriše vse črke, a bo lažje, če jih enostavno pobrišeš v podatkih.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tinahawlina/APPR-2014-15/commit/159de165623a4eb1e53324f66adb00083e5efb37#commitcomment-9359420>
> .
>


Right now, this scaler gives us the potential to get 100 out. Since we are dealing with percent heights in the css, we should have it only return a max of ~75 so it doesn't overlap with the value. (That's what the previous scaler function did)


You could just return `k` here – variadic parameters have type `[K+]`, not `{K+}`.


Вот так было изначально.
```js
function PagesLoader(from, count, loadFn, errorFn, endFn) {
	var queue, i = from,
		len = Math.min(aib.lastPage + 1, i + count),
		queue = new $queue(2, this.loadPage.bind(this), endFn);
	this.queue = queue;
	this.loadFn = loadFn;
	this.errorFn = errorFn;
	while(i < len) {
		queue.run(i++);
	}
	queue.complete();
}
```

`i` дублировала аргумент функции `from`, который больше нигде не использовался.


missing the () on the function call


This with app.iml is a default Android Studio crap, that specific you can
ignore if you'd prefer otherwise. I can .gitignore it and keep my local
copy, no sweat.

D.

On Tue, Jan 20, 2015 at 8:53 PM, kost <notifications@github.com> wrote:

> @dkorunic <https://github.com/dkorunic> This is mistake, I guess?
> Personally, I prefer relative path over absolute one in this case...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kost/NetworkMapper/commit/48bea36181c98ff88d5b31b3ff61a1ff822516fa#commitcomment-9362791>
> .
>

--
NAME:Dinko.kreator.Korunic   DISCLAIMER:Standard.disclaimer.applies
ICQ:16965294        JAB:kreatorMoo@jabber.org        PGP:0xEA160D0B
HOME:http://dkorunic.net    QUOTE:Eat.right.stay.fit.and.die.anyway


Gems usually don't have a Gemfile.lock. We should have all dependencies spelled out in the .gemspec file. Gemfile should just be:

```Gemfile
source 'https://rubygems.org'
gemspec
```


Gosh I can't spell!


>If you use the Encryption class, you must set the an encryption key.

I think that comma is unnecessary, it should be

>If you use the Encryption class you must set the an encryption key.

or maybe even

>When using the Encryption class you must set the an encryption key.


Ах, лол. Я это скопировал с `loadPages` и там были замыкания.


Ну, и `queue` зачем-то переопределялось. Сакрального смыслу не понял.


@AlwinEsch okay, cool. I honestly didn't even have a clue what any of these dialogs were supposed to do.


Got it, no biggy. Thanks again!


@gaganarora1 just install RMySQL from github with `devtools::install_github("rstats-dbi/RMySQL")`


Closes #6 for good.


looks like a merge conflict got committed before it was resolved?


This was a change to matter.js, not to a local file. Be careful.


I think so. And more importantly, it sounds like you understand the issues involved, so I'm confident this is working as intended now. Thanks for clarifying.


ctrlsmart.com


Translation of RNA was done in prot.py although code isn't pretty


Thank you for your ongoing support of this project! I really appreciate it!


Good catch here.


I saw this on xda, I already have a material dark theme in my rom that we've customized but I'm curious as to documents ui if you were able to do that and also the pop up dialog boxes, similar to the rebooting and upgrading boxes if you were able to make them black? I'll attach a SS of what I'm talking about. I keep looking but can't seem to find where they are calling for the colors. Any help would be appreciated. Second SS is of stweaks which is fine itself BUT the teal bar at the top is in many apps and I can't seem to find the overlay for it either Right under the cpu tab you can see it
![screenshot_2015-01-19-21-14-29](https://cloud.githubusercontent.com/assets/6742515/5811164/3d6e560c-a020-11e4-8935-d7fdb6d5ef8e.png)

![screenshot_2015-01-19-21-12-18](https://cloud.githubusercontent.com/assets/6742515/5811154/fd2704d6-a01f-11e4-9de3-9e7d44db8c39.png)


Maybe be need te let the user choose a new one?


Thanks, I will submit the patch later.


Nice to see some serious documentation getting written.


Maybe we need to let choose the user a new one?


![image](https://cloud.githubusercontent.com/assets/10607120/5811175/83bc41c4-a08d-11e4-980e-d750e121a957.png)
https://g.co/doodle/pahjk2


Wait, if the blades weren't handled correctly, would they just fall off and throw like a boomerang and slit Legal Eagles' chests? :o


Hi Josh. You may like to check out https://github.com/yawaramin/lambdak

I'm trying to build a true anonymous function expression in Python out of lots of little lambdas, completely avoiding `exec` and `eval`. For an introductory blog post which walks through a simpler version, see http://yawar.blogspot.ca/2015/01/expressive-functional-programming-with.html


Indeed. Furthermore, AD can't tell me about shop purchases (which is what this code is used for).


I think that this addition is very useful but does not belong in the core. However you can replace the default /oauth/me functionality by using the endpoint filter. I had added a gist of how to replace the core `me` method with the the above example https://gist.github.com/justingreerbbi/8b0342b677c48dc5109d


yea, it will inherit from the parent. should be good to remove.

https://github.com/gempesaw/Selenium-Remote-Driver/blob/7cc2771c2fd077e187c099ffb90416f556fc5472/lib/Selenium/Remote/Mock/RemoteConnection.pm#L12
https://github.com/gempesaw/Selenium-Remote-Driver/blob/7cc2771c2fd077e187c099ffb90416f556fc5472/lib/Selenium/Remote/RemoteConnection.pm#L24


Better to use bigint data type (Long in java) for employee Id


Ok. Your are right, but i think i have to change the signature of the method to "public Double getTotal();" and make Purchase order to implement it, or better, make IOrder to extend IDiscount. What do you think about it?


Right. I'm not sure how useful it is to emulate the exact limitations/bugs of Vim, but anyway. In order to do that and keep Unicode letters working `isKeyword()` needs a check like `return c < 256 ? keywords.contains((int)c) : Character.isLetterOrDigit(c);` and tests for a character like 'Ź' (U+0179).

Unless I'm missing something, the upper bound of the loop should be 256 (to get characters 0-255).


뒤로가기 버튼 이벤트 리스너 추가


Thank God, README is finally readable now


## This is a Markdown File


微商代理授权查询系统-微商系统™官网
http://weishang.hengyuejiaqi.com/


This is an example script from the [Bitcoin wiki](https://en.bitcoin.it/wiki/Script)


I flipped the order of the two first arguments because tests report the second parameter as expected and the first parameter as actual.


Sorry, should have mentioned before--this one is also opposite order.


The bug has been removed. Now just working on pushing in short shifts and removing the delay implemented in the 'c' file.


Makes sense. I think we can override the [parent's `remote_server_addr`](https://github.com/gempesaw/Selenium-Remote-Driver/blob/7cc2771c2fd077e187c099ffb90416f556fc5472/lib/Selenium/Remote/RemoteConnection.pm#L16) and set a default for it all in one go with `has`.

```perl
has '+remote_server_addr' => (
    is => 'lazy',
    default => sub { 'localhost' }
);
```

I think this should accomplish the same thing, but make the behavior more clear.


merge first commit


Yes, but I think it's set now.


哈哈，不错


fast-forwarding develop to borshop-integration-1052-bugfix/fix-hardcoded-crashdump-path = ba338671


Primera version


// is a problem


It would be nice with a template here. This version number will change a lot and it would be cool if that was just part of a build step.


Updated gh-pages


@marksantcroos thanks, this is now fixed in open-mpi/ompi@9f80aa2d2891d94ba3a3b7870d4a58b1871e50f6


+1 to that. We're working on the build process this week.


@kylegalbraith I reverted this commit because I realized this work wasn't for this story


Cool! Also https://github.com/iojs/io.js/issues/500#issuecomment-70595610 (just for reference)


Why is Offer extending from an Item ?. Probably I'm misunderstanding something here, but I was expecting Offer to be a composite, since according to the exercise it may contain Items, but also other orders.


I wonder about shelling out to ag to get its help info and parsing that... But I worry that's not the right way either


This should be
`scrollTop  = (typeof top === 'undefined') ?contentViewport.scroller.scrollTop : top;`

otherwise it does not handle zero correctly


I applied your suggestion in #22 does this need to be updated for that change?


Warning: the for introduced in Java 5 is prefered. Something like for (Item item: items.values()) {}.


It would be nice if we could still accept them with out registering each option with `add_option`, but blindly pass them along to `ag`.

Since we can't -- if we wanted to automate it we'd have to shell out to `ag` even when printing the help text :flushed:


We are using the concept of Relational Database models right?.
So let assume a real time scenario, do salary element exists if no any employee are there, Here I can directly create salary without creating its corresponding employee which is not correct.
So according to this design, there must be department and designation before employee , and then there must be employee before salary.  (See the concept of foreign key)


Put the contents of javascript.txt Javascripts section of your Board Template, likewise put the below the board.txt into the Below the Board section.


I don't understand the question.


Thanks for this! I'm in the midst of a lot of travel for work, but I'll give it a try and hopefully merge it soon!


你好，能不能把这里的corpus.bin的语料库给个下载地址？谢谢，赠人玫瑰，手留余香~~~


Right on! (thumbsup)


Scheduled messages are not yet delivered hence they are not supposed to be on the iterator...

this is a rejection


Yeah I was able to move this back into the method itself since now that we're "fork"ing instead of "spawn"ing, the subprocess has access to global `Configuration`.

I thought it was also a good idea to load the config value up here instead of below (in the branch logic) so that if we make future changes to how this subprocess works it will be harder to introduce a bug around loading the config. (For example, the default subprocess method on Windows is "spawn", so this part of the application would definitely need some changes to support Windows.)


Muy bien! ya empezaste con el pie derecho :+1:


Nice :)


OK thanks. Please review the pre-work doc and when ready raise a github issue in this repo as well as add a [README with GIF walkthrough](http://courses.codepath.com/snippets/intro_to_android/prework#heading-4-submitting-the-project) as outlined there. Ping me back here afterwards. Thanks.


That's the Gemfile. The lock probably just needs to be removed from git added to .gitignore


This class should not be dependent on knowing the internal details of the validator. All it should know is that the validator needs to be called to do its thing. And this validation could be more than simply the start date.
Does this make sense? :)


Yup makes sense - I haven't even referenced it yet.


Your commit only changes the lock file.


That makes sense :)


i'm orgasaming


Muy bien Eduardo!  :+1:


Muy bien!  :+1:


We actually need to be this precise:

https://github.com/GoBoundless/versioned_record/blob/master/versioned_record.gemspec#L21-L32


While it's nice to have the comment that 0 is life stage and 1 is gender, I think it would be clearer if you used words. i.e.: at the top of the class, add

```ruby
LIFE_STAGE = 0
GENDER = 1
```

...And then change this code to be, e.g.:

```ruby
if a.context_labels[GENDER].to_s == b.context_labels[GENDER].to_s && a.context_labels[LIFE_STAGE].to_s == b.context_labels[LIFE_STAGE].to_s
```

...This having been said, I'm still _slightly_ wary of how ... hard-coded this feels. Anyway, I think it will do.


Ugh, exceptions make C++ code so hard to follow. However, after digging around through ```decodeRow()``` and ```decode()``` it seems you're quite correct i.e. result.empty() will never be true in this method.

Given the changes you've made. Could you please rename this method to ```extractUpc()``` (or something similar), because it is now expected that this method always return a result (or throws an exception).


`ceylon.ast` calls this `ImportFunctionValueElement` (I assume I chose the name based on `FunctionValueAlias`) – should I rename it?


Bien, aunquetrata de hacerlo en en Java :radio:


You just said, "Gems usually don't have a Gemfile.lock. We should have all dependencies spelled out in the .gemspec file. Gemfile should just be:

source 'https://rubygems.org'
gemspec"

When I said "That's the Gemfile" I meant the Gemfile already does say that. You're commenting on the lock file. The lock file was already in source control and was updated when I did a bundle, which is what caused this. I've already removed it


Should we pass the "kind" parameter to process.connections() to filter on network connections?  These are the available options ... by default it uses "inet"

  Kind Value      Connections using
        inet            IPv4 and IPv6
        inet4           IPv4
        inet6           IPv6
        tcp             TCP
        tcp4            TCP over IPv4
        tcp6            TCP over IPv6
        udp             UDP
        udp4            UDP over IPv4
        udp6            UDP over IPv6
        unix            UNIX socket (both UDP and TCP protocols)
        all             the sum of all the possible families and protocols


With regard to the duplicated keyCredential documents, the best way to explain it that I can think of is this:

the keyCredential which is inside the masterSigned element is the "intermediate form" of the keyCredential.

The outermost keyCredential element is the "public form" of the keyCredential

The sequence of creating a keyCredential then becomes:

* Create the intermediate form
* Have the master credential sign the intermediate form
* Create the public form by combining the master credential and the master-signed intermediate form into a credential document


Published to http://vui-docs-internal.s3-website-us-east-1.amazonaws.com/19_6348df879b8bf26e66cace51d9d84a3fe5e3f488


@lazedo where in the database do this design documents/views go?  I can't find them


@neilself this doesn't work, after an additional update to change the name of the function you edited in another place in code.

My guess is it has to do with the main code path being used to calculate statistics being different, although I'm honestly not familiar with the code. Generally the trigger for calculating statistics is a publish button through the admin side.

In testing, I reported 5 matches which were ended in a 3-0, but all 5 showed up in the staff admin panel. When I published the match in the admin panel like we normally do, all of those matches were still included in the tiebreakers for both teams, and the individual player records.


installing this package "liblz4-tool" will certainly fix the problem let me know I have a fix if this doesn't work


Cool thanks @Garcia98


Ah, so this is the big change, right? Can you describe in high level what you've added here and why? Maybe have it in the commit message?


Yes I noticed that and it's fixed locally but not pushed yet... still testing interoperability with other libs.


No to usun podkreslenie bocie, skro nie jest juz private.