Yeah - sorry, I should have been more clear about what the change actually does; it doesn't actually affect the parsing, which will require an update to the config files at https://github.com/GoogleCloudPlatform/fluentd-catch-all-config.  We've decided to hold off on doing that for a bit to polish our support for structured logs, although I might do an interim update to at least extract 'time' and 'severity' where applicable.

You can also do this yourself by editing the config files (in /etc/google-fluentd/config.d).  http://docs.fluentd.org/articles/in_tail is probably the most applicable documentation.  You'll see with a couple exceptions we are generally using 'format none' but you can change that to one of the supported formats or a regex.  "time", "severity", and "message", are treated specially; if you extract other keys they will also be ingested as structured data with the 0.2.3 plugin.

We can definitely continue this on Stack Overflow as well.  I'm working on getting an "official" google-cloud-logging tag set up so just use google-cloud-platform for now.


Yes, it probably is. This was a bit improvised. We need to revisit this code after all the issues are fixed and the site is live.


This is the startup scripts on a basic Wheezy system:

```
/etc/rcS.d/S01hostname.sh
/etc/rcS.d/S01mountkernfs.sh
/etc/rcS.d/S02udev
/etc/rcS.d/S03keyboard-setup
/etc/rcS.d/S04mountdevsubfs.sh
/etc/rcS.d/S05hwclock.sh
/etc/rcS.d/S06checkroot.sh
/etc/rcS.d/S07checkroot-bootclean.sh
/etc/rcS.d/S07kmod
/etc/rcS.d/S07mtab.sh
/etc/rcS.d/S08checkfs.sh
/etc/rcS.d/S09mountall.sh                          <- this is where fstab mounts are mounted
/etc/rcS.d/S10mountall-bootclean.sh
/etc/rcS.d/S11procps
/etc/rcS.d/S11udev-mtab
/etc/rcS.d/S11urandom
/etc/rcS.d/S11zfs-zed
/etc/rcS.d/S12networking
/etc/rcS.d/S12zfs-import                           <- and here we import the pool(s)
/etc/rcS.d/S13rpcbind
/etc/rcS.d/S14nfs-common
/etc/rcS.d/S15mountnfs.sh
/etc/rcS.d/S16mountnfs-bootclean.sh
/etc/rcS.d/S17kbd
/etc/rcS.d/S18console-setup
/etc/rcS.d/S19bootmisc.sh
/etc/rcS.d/S19plymouth-log
/etc/rcS.d/S19x11-common
```

This is just the S runlevel. After this, the multiuser runlevel scripts are run:

```
/etc/rc2.d/S01motd
/etc/rc2.d/S01zfs-mount                            <- this is where we mount legacy filesystems and volumes (now).
/etc/rc2.d/S13rpcbind
/etc/rc2.d/S14nfs-common
/etc/rc2.d/S16rsyslog
/etc/rc2.d/S16virtualbox-guest-utils
/etc/rc2.d/S16zfs-share
/etc/rc2.d/S17acpid
/etc/rc2.d/S17atd
/etc/rc2.d/S17cron
/etc/rc2.d/S17exim4
/etc/rc2.d/S17ssh
/etc/rc2.d/S19bootlogs
/etc/rc2.d/S20plymouth
/etc/rc2.d/S20rc.local
/etc/rc2.d/S20rmnologin
```

It depends on where zfs-import and zfs-mount is run on CentOS6. The purpose of the new script was so that we can run it as early as possible. I'm not even running as early as I can, I just noticed.

It should be possible to run it between S07checkroot-bootclean.sh (which cleans up /var/lock which we need 'intact'/'unchanged' after zfs-import and S08checkfs.sh (which checks all filesystems in /etc/fstab - because I doesn't now, I need the fsck column to be 0).

So if I can get it to run at the right level, maybe this ISN'T necessary after all. I'll do some tests (it's difficult to hit the target, because Wheezy isn't actually using those links - it uses startpar to run things in parallel and getting the dependencies right have been a pain).

How does it look like on CentOS? Do you have any command line examples to show how to create them correctly (in the right 'place'/'level')?


Ok, I can live with that compromise.


why the commented out code?


This is exactly what I was testing. Here it caused successful SMP boot with 4 cores :)

But things was slowed down (hearbeat LED at least) until I added some code for setting up prescaler. I was doing a quick test by adding it in bcm2836_platsmp.c because there is local_mbox pointer available, but I was about to move in board_bcm2385.c later this evening and then produce a commit.

Seeing you are currently working around this triggered me to write you right now to ensure avoiding duplicated work :)

FYI I did something like this:

+#define ARM_LOCAL_CONTROL 0x0
+#define ARM_LOCAL_PRESCALER 0x8

+               writel(0, local_mbox + ARM_LOCAL_CONTROL);
+               /* timer prescaler 19.2MHz */
+               writel(0x80000000, local_mbox + ARM_LOCAL_PRESCALER);


Should it get checked whether it really got used together with -o?

`exportScoreParts = parser.isSet("export-score-parts") && parser.isSet("o");`


Better name: doHousekeepingByPreservingCacheItemsWithURLs ?


And coming back to the issue this means that ``rand(::AbstractRNG, ::Type{Char}) `` should indeed return a *random unicode scalar value*.


This is just a super rough draft - I'll fix it as I rewrite it (most it will probably change)


The loop in the new code is from 0, which is GEANY_FILETYPES_NONE and filetypes_get_conf_extension(ft) returns the "common" extension for it.


use `// +build !windows` to be available on anything else


Nope, you didn't add the country, but just the country id. You find the country's name in table configuration as PS_SHOP_COUNTRY


I think computer might be better in this case. Machine is ambigous (it could be a car engine, etc), while computer is explicit.


Thanks :wink:


ok


typo encounterd


Why 'profiles'? User has one profile, and you can have resource :profile


or even
`
if  ((exportScoreParts = parser.isSet("export-score-parts") && !parser.isSet("o"))
      parser.showHelp(EXIT_FAILURE);
`


This drupal_alter happens to unset agents from the list, so the "is the list empty" question should be asked only after the altering.


the original is correct here. the code snippet starts with `42` and gets `"42"`. So, "`"42"` comes *from* `42`", as originally written.


good catch! :)


makeAllCacheFilesPerformBlock ?


This macro is not needed anymore, I change all occurrences to use `estrndup`


I can add a check for the "filetypes." prefix and just skip the whole loop in case the prefix is different.


As I mentioned previously, computer might be better because it is explicit.


, at the end is invalid


For what purposes you added hidden field id?


At this point i tried to add two tests to test the endFrame length.
but as fare as i found it is not possible to change the pixel_count after the initialization of the output.
and i did not know how to 'reinitialize' the output with new options..


Exactly, it will only match once - I didn't add the break because I'd have to duplicate the g_free() below but maybe it will make things clearer.


shouldn't we drop the whole cache in this case (infodict, ...) ?


I was thinking that the `doUnlock` function needs to be called every single time because it keeps track of how many times it has been called, and "after" `ids.length` times, it will call the actual warlock `unlock` function.  I was also thinking that we really don't need to check for an errors in the `inactive` callback because, we want to release the lock regardless of an error, and either way, the warlock ttl will expire in 2 seconds anyway.


This commit removes `mTxSetFetches`, since the fetching is now the responsibility of `PendingEnvelopes`. Since previously `mTxSetFetches` was also used to hold the proposed sets into the cache, and that that responsibility remains with `Herder`, this hunk introduces `mProposedTrackers` just for that purpose.

It's really not something that can be moved to a different comit.


Instead of adding these files just modify your controller action to include a render statement:
```
def batch_create
  render json: {}
end

```


typo: fileNonExistendOrDeleted  -> fileNonExistentOrDeleted


It was democracy, me and sam started a revolution.


But it was sam's idea, don't shoot the messenger Bro


else set the homeDomain to empty string - but then you could also make the column "NOT NULL" as it's mandatory in the account object


@zewa666 @EisenbergEffect Was this intentionally changed?  Doesn't appear to do anything.


Looks good.


I disagree. It doesn't flow nicely. Something is not right with *not* including "that are". I've revised it to be more clear, which I will submit in a new PR.


Commented line can be removed.


I don't get it... is it so that when the original test fail, it returns something like this ```[ 4, [] ]```?


Removed this in next commit.


Java is a shit language; I will not fight you.


should use string32 to match how options set the field


Just added the title of the course.


Когда сделаешь змею отдельным классом, не потребуется и глобальная переменная.


I cannot find any usage of this except [here](https://github.com/jenkinsci/backend-jpi-create/blob/5e01014e9507029113556dabbcb8dae129824e09/src/main/java/org/jenkinsci/backend/jpicreate/WebAppMain.java#L25). I would have expected Jenkins core to configure it, so that I could write static files as `*.html` rather than `*.jelly`.


why did you change it? See https://github.com/dset0x/invenio/commit/bcc52389dacb154b7565206a44f662227e420063#diff-a367ebdf28b638dc9b793ec120ef51e9R70


comment


oh, wow, that was unexpected.


Held ég sé kominn í gegnum þetta -- þarf greinilega bara að loka augunum og vita að þetta er tag, svarar rétt við isinstance() til dæmis. Alltaf að læra :)


Yes


Test coverage is not equal to test correctness.
1. the unit test aims at the internal helper function, which doesn't strictly define the behavior of the package's public api.
2. The test is not correct implemented. The root config is intentionally random, and so we would like to test that the resulting signer is nil, the resulting bool of input well-formed-ness (which can enjoy some more documentation on its purpose) is false, and the error is nil.
3. What I suggests is to test NewSigner(), which would provide much better coverage about 80%~90%.


Aaa


Yeah, it's the config menu only - having the filetype menu first (which corresponds to the alphabetic order) seemed a bit strange so I added this.


Yup


the enum values have to be named like
MEMO_TYPE_NONE, etc
as C++ defines those in the current namespace.


So, why are all the tests using lists then?


My reply? Or the limitation? :smile:

I was surprised by this as well. I had grand visions for `security_group_rule` but it didn't live up to my dream(s).

If you see otherwise, I'm all ears


Credit: @Xiledria -- see PR #14641 and issue #13964.


How about: https://github.com/popcornmix/xbmc/commit/d95884291d18f87483781f2a9fa6bc6ab65a17e6


This is not well formed (extra unclear closing tag)


Top of the hierarchy for all GoodGuys and BadGuys in the game.


And this is surprising. I would have expected that if I added `new StaticViewFacet("html")` then a request to `<st:include it="${something}" page="help"/>` would load either `help.jelly` or `help.groovy` or `help.html`. But it seems like I would need to request the view as `<st:include it="${something}" page="help.html"/>`, which seems to defeat the purpose. Am I missing something?


Thanks! But you probably meant @boonebgorges :)


this commit broke first-run launch configuration -- please fix ASAP


value is one of the cases where iirc jquery being very useful for x-browser compatibility: so $(e.target).val() might be better


Wrong stack trace. The bug pertains to ballot timers.

It's probably not worth tracking further since they are gone in your scp branch.

```
>	stellar-core.exe!stellar::Slot::bumpToBallot(const stellar::SCPBallot & ballot) Line 235	C++
 	stellar-core.exe!stellar::Slot::processEnvelope::__l2::void <lambda>(bool)::__l8::<lambda>(bool valid) Line 121	C++
 	[External Code]
 	stellar-core.exe!stellar::HerderImpl::startBallotTimer::__l2::<lambda>(const std::error_code & __formal) Line 542	C++
 	[External Code]
 	stellar-core.exe!stellar::VirtualClockEvent::cancel() Line 339	C++
 	stellar-core.exe!stellar::VirtualTimer::cancel() Line 377	C++
 	stellar-core.exe!stellar::VirtualTimer::~VirtualTimer() Line 366	C++
 	[External Code]
 	stellar-core.exe!stellar::HerderImpl::ledgerClosed() Line 920	C++
 	stellar-core.exe!stellar::HerderImpl::valueExternalized(const unsigned __int64 & slotIndex, const xdr::xvector<unsigned char,4294967292> & value) Line 666	C++
 	stellar-core.exe!stellar::Slot::attemptExternalize() Line 435	C++
 	stellar-core.exe!stellar::Slot::advanceSlot() Line 693	C++
 	stellar-core.exe!stellar::Slot::processEnvelope::__l2::<lambda>(bool valid) Line 190	C++
 	[External Code]
 	stellar-core.exe!stellar::HerderImpl::validateValue(const unsigned __int64 & slotIndex, const xdr::opaque_array<32> & nodeID, const xdr::xvector<unsigned char,4294967292> & value, const std::function<void __cdecl(bool)> & cb) Line 249	C++
 	stellar-core.exe!stellar::Slot::processEnvelope(const stellar::SCPEnvelope & envelope, const std::function<void __cdecl(enum stellar::SCP::EnvelopeState)> & cb) Line 206	C++
 	stellar-core.exe!stellar::SCP::receiveEnvelope(const stellar::SCPEnvelope & envelope, const std::function<void __cdecl(enum stellar::SCP::EnvelopeState)> & cb) Line 36	C++
 	stellar-core.exe!stellar::HerderImpl::processSCPQueueAtIndex(unsigned __int64 slotIndex) Line 896	C++
 	stellar-core.exe!stellar::HerderImpl::processSCPQueue() Line 867	C++
 	stellar-core.exe!stellar::PendingEnvelopes::checkReady(std::shared_ptr<stellar::PendingEnvelopes::FetchingRecord> fRecord) Line 231	C++
 	stellar-core.exe!stellar::PendingEnvelopes::fetch(const stellar::SCPEnvelope & env) Line 253	C++
 	stellar-core.exe!stellar::PendingEnvelopes::add(const stellar::SCPEnvelope & envelope) Line 43	C++
 	stellar-core.exe!stellar::HerderImpl::recvSCPEnvelope(stellar::SCPEnvelope envelope, const std::function<void __cdecl(enum stellar::SCP::EnvelopeState)> & cb) Line 846	C++
 	stellar-core.exe!stellar::Peer::recvSCPMessage(const stellar::StellarMessage & msg) Line 355	C++
 	stellar-core.exe!stellar::Peer::recvMessage(const stellar::StellarMessage & stellarMsg) Line 250	C++
 	stellar-core.exe!stellar::Peer::recvMessage(const std::unique_ptr<xdr::message_t,std::default_delete<xdr::message_t> > & msg) Line 161	C++
 	stellar-core.exe!stellar::LoopbackPeer::deliverOne::__l8::<lambda>() Line 175	C++
 	stellar-core.exe!asio::asio_handler_invoke<void <lambda>(void) >(stellar::LoopbackPeer::deliverOne::__l8::void <lambda>(void) & function, ...) Line 69	C++
 	stellar-core.exe!asio_handler_invoke_helpers::invoke<void <lambda>(void),void <lambda>(void) >(stellar::LoopbackPeer::deliverOne::__l8::void <lambda>(void) & function, stellar::LoopbackPeer::deliverOne::__l8::void <lambda>(void) & context) Line 39	C++
 	stellar-core.exe!asio::detail::completion_handler<void <lambda>(void) >::do_complete(asio::detail::win_iocp_io_service * owner, asio::detail::win_iocp_operation * base, const std::error_code & __formal, unsigned __int64 __formal) Line 67	C++
 	stellar-core.exe!asio::detail::win_iocp_operation::complete(asio::detail::win_iocp_io_service & owner, const std::error_code & ec, unsigned __int64 bytes_transferred) Line 46	C++
 	stellar-core.exe!asio::detail::win_iocp_io_service::do_one(bool block, std::error_code & ec) Line 405	C++
 	stellar-core.exe!asio::detail::win_iocp_io_service::poll(std::error_code & ec) Line 197	C++
 	stellar-core.exe!asio::io_service::poll() Line 84	C++
 	stellar-core.exe!stellar::VirtualClock::crank(bool block) Line 236	C++
 	stellar-core.exe!stellar::Simulation::crankAllNodes(int nbTicks) Line 174	C++
 	stellar-core.exe!stellar::Simulation::crankUntil(const std::function<bool __cdecl(void)> & predicate, std::chrono::duration<__int64,std::ratio<1,10000000> > timeout) Line 286	C++
 	stellar-core.exe!hierarchicalTopo(int nLedgers, int nBranches, stellar::Simulation::Mode mode) Line 85	C++
 	stellar-core.exe!____C_A_T_C_H____T_E_S_T____97() Line 113	C++
```


:+1: verboser, but clearer overall


@cahrens Here's the simpler version that doesn't provide the diff feedback.


> My reply? Or the limitation? :smile:

The limitation, and that all tests seem to be using lists. I'm a bit confused.


I remember thinking in my head to type this flag, and then... didn't


wait, I'm sorry, I may be confused and commenting on the wrong things. For security group _rules_, it's a single CIDR IP. For Security groups with in-line rules, it should be a list. My apologies! I'll clear up the documentation.


Maybe we should just rename this variable to m_openFileCount


I wasn't talking about `inactive` errors but `Job.get` errors (line 191) which will prevent `doUnlock` to be called and it won't reach `ids.length` ever, however there warlock's timeout comes to help, but this is just an escape to depend on that timeout ;)


Right...
> The commands are also allowed to return zero elements, and the client should not consider the iteration complete as long as the returned cursor is not zero.

I guess the while() look makes sense then. I actually would like you to factor out the while(1) loop and use it for all the *scan* tests. Something like this:
```
sub scan_test {
  my ($method, @args) = @_;
  my $guard = 1000;
  my ($res, %res);
  while ($guard--) {
    $res = $redis->$method($res->[0] || 0, @args);
    $res{$_} = 1 for @{$res->[1]};
    return sort keys %res unless $res->[0];
  }
  return "GUARD";
}
```

Also, sorry for being so rash about the while(1). I didn't understand how you terminated it the first time.


Thanks,
fix here
https://github.com/apache/parquet-mr/pull/191

On Wed, May 6, 2015 at 6:27 AM, du00cs <notifications@github.com> wrote:

> This will cause incompatible with jdk 1.6 before,
> ReflectiveOperationException is introduced in jdk 1.7
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/parquet-mr/commit/52f3240d90f2397cd1850ab11674ba08a0ecb2a0#commitcomment-11065301>
> .
>


You don't want to duplicate these lookup table. So have a single map of version -> pretty name, and use it both in  _getOptions and _getOptionRealValue


Hi, just to understand the update here, so everything is supposed to be internal references now?


Please double check on version number since it could be still part of 2.4?


ugh, even _that_ statement isn't right. Just ignore me, I'll review and update the documentation :smile: . Other than it should be a list, was there something else? Thanks!


maybe should be opaque[32] instead?
string<32> has a 4 bytes length in it, so it's really 36 bytes
if we want variable length - why 32?


This will work but it has become pretty elaborate, how about you have a map of regexes -> pretty names and check each option against the list for matches?


we have typedefs "Hash" that you should use for HASH and RETURN types


Yes. Tweaking the LSB header made it work as intended (without that extra code). I see that you're reviewing again, so I'll push that part later.


Argh. Ok. I fixed and re-pushed.

On Wed, May 6, 2015 at 2:46 PM, Raffaele Viglianti <notifications@github.com
> wrote:

> This is not well formed (extra unclear closing tag)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/charlescarter/sga/commit/70ba9b51c5887d8d78823a8e7ad0ac5c0e3ea81e#commitcomment-11072161>
> .
>

--
________________________________________________________

Charles Cuykendall Carter
Assistant Curator
The Carl H. Pforzheimer Collection of Shelley and His Circle
The New York Public Library, rm. 319
Fifth Avenue and 42nd Street
New York, New York 10018

(212) 930-0717

charlescarter@nypl.org


Thanks for catching these! Apparently my preview tool, "Marked 2.app," led me astray on the links. :)

Did you notice any others that were wrong?


See also https://github.com/TrinityCore/TrinityCore/commit/1161295f4f933661ea9738e0c57049a1e38ec8f2 for the merge, with precise header description by @jackpoz


tabstabstabstabs


@boone Hey dude, cool screenname


Do you have a clue why?


I know it's a rare chance, but if somehow the variable is changed and we get a non-existing file handle, this will return default-initialized FILE*, which will be nullptr and fclose(nullptr) will probably segfault :-( To be sure nothing goes wrong, we should check against it.


This should also significantly reduce memory footprint because shares of given job are stored in binary form instead of hexa.


These `<TO_BE_ADDED>` likely need to be replaced before submitting PR, otherwise, you'll need to submit another one.


In this case, we have a type that ideally is an IterableIterator. We want to do a structural dive to retrieve a particular type T from inside the IterableIterator. But there are two places that T could appear. One is the spot where it belongs in an iterable, and one is the spot where it belongs in an iterator. Ideally in such cases, the two candidates for T would be the same, so it is sufficient to just pick one.

Also, the type could just be an iterator, or it could just be an iterable, or any supertype of IterableIterator. In this case, it is possible that T could appear only once in the type, but we are not sure which spot it will be at, nor do we care.

I suppose the idea is that getElementTypeFromIterableIterator makes the best effort to find a relevant appearance of the type T inside the input type.


I did a bisect and before this commit, first-run worked correctly.  I tested by removing my mixxx.cfg for each rerun of the bisection.


Fixed.


Hmm, all the callers just pass undefined. I'll remove the parameter


Good call.  Simplified.


Weather Permissions


This sentence seems like it needs to be replaced.


No worries, that clear things up for me! Thanks @catsby.


I like it-- thanks. If you wanted to list the reports that are present if download_links.length > 1, you could add on download_links.attrs('href') in the message for that assertion.


Why was this changed? Is it an artifact of the change above?


but it is empty string by default right?


@orangejenny
make sure you take this step into account as you are redoing the sign up process.


called them error_file and error_line, because they come from ERR_get_error_line, not SSL_get_error, which also exists for some reason :-P


I would not use the `bind` word here since it's overloaded with `service binding` which is still a thing in that release.

Maybe we can use `connect to` or `access`?


rax retry tests


I wanted to change as little as possible compared to the old maven build script, but I already have another commit here that will change it to 1.7 once this is merged. We still need 1.7 unfortunately...


really? think they wont get what this is for?


w


@azevedo-252 Why do you assume he deleted the comment just because he felt like it? As he said, I told him to, because it was indeed a copy&paste error. It was not suposed to be there in the first place

As for the values in the comment, neither of us could figure out what they were suposed to mean, which is why we skipped them at the time

Anyway, as he said, this needs to be revised later. It was deploy day at the office, which we all know how it goes :shipit:


> do these inherit and merge?

Would also like to know


Fixed


This will run the first test and the first test only, forever.

How about:
            for (i=0; nr_loops==0 || i<nr_loops; i++) {

which is a little easier to read?


@mitchellh This is why we can't have nice things.


Add WebKit.framework


This will run the first test and the first test only, forever. Oh well.


LGTM


May I ask what your use case is? I don't object to this, just curious.


Right, sorry. I misread.


This seems repeated from `index.html.md.erb` ?


Only add the screen, if there isn't a plugin with the same name.


thx a lot! very interesting.


Make sure this output matches exactly what the latest CLI output is when issuing the command. Thanks.


highly appreciated


changed the extract too


Missing '?showinfo=0&rel=0' at the end.
Adding this prevents related videos from being showed once the video has finished.


The `goimports` tool will do this: `go get golang.org/x/tools/cmd/goimports`


Yeah I like this more too. Shall I send an amended pull request for this?


Well, I could add that but only as an additional option to this menu (and I'm not sure we need two ways to open the config files). The thing is if the "open current filetype configuration" was the only option, this would require that you have the given filetype open which doesn't have to be the case. Forcing users to open (or create dummy) file with the given filetype just to edit the configuration is a bit silly...

(off-topic part)

...which is exactly something we force users to do in the "Set build commands" dialog. Users can't create a project and set up the commands immediately. They have to first create files with given extensions and then select file with the given extension, open the dialog, edit the properties and close it - and repeat this procedure for every single filetype! Crazy.

One of the things I'd like to do is to add combo boxes with filetype lists for the filetype-specific parts of the dialog so users can change all the filetype-specific commands in one go.


Looks good to me.


I agree. Feel free to start working on one. It is also important to provide methods such as ItemFactory.makeCorpse(Creature).

-----Original Message-----
From: "Alexander" <notifications@github.com>
Sent: 5/6/2015 3:04 PM
To: "neurolytic/dungeon" <dungeon@noreply.github.com>
Cc: "Bernardo Sulzbach" <mafagafogigante@gmail.com>
Subject: Re: [dungeon] Major commit that enables that implements itemdropping. (026581d)

Maybe we need an ItemFactory instead of the Item(ItemBlueprint) constructor.
—
Reply to this email directly or view it on GitHub.


sure :)

On Wed, May 6, 2015 at 9:03 PM, Stephen Bates <notifications@github.com>
wrote:

> Yeah I like this more too. Shall I send an amended pull request for this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sbates130272/mbw/commit/1ff382d18946a15615be1771f98cb603a90fb07a#commitcomment-11072494>
> .
>


this breaks the file.recurse function:

```
----------
          ID: salt_vimfiles_salt
    Function: file.recurse
        Name: /usr/share/vim/vimfiles
      Result: False
     Comment: An exception occurred in this state: Traceback (most recent call last):
                File "/usr/lib/python2.7/site-packages/salt/state.py", line 1588, in call
                  ret = self.states[cdata['full']](*cdata['args'],
                File "/usr/lib/python2.7/site-packages/salt/states/file.py", line 2289, in recurse
                  manage_file(dest, src)
                File "/usr/lib/python2.7/site-packages/salt/states/file.py", line 2133, in manage_file
                  source = salt.utils.url.escape(source)
              AttributeError: 'module' object has no attribute 'escape'
     Started: 21:01:15.482366
    Duration: 85.073 ms
     Changes:
----------
```


If this is an empty struct, it should be

```
type ResponseEx struct {}
```

Given that it's exported, it should have a doc string. Given that it's an empty structure as a type, there should be some comment explaining why it's needed.


I often do IO performance testing (for example on NVMe SSDs and RDMA NICs) and am keen to see how the IO DMA accesses into DRAM impact on the CPU cores ability to access the same DRAM. By running your program in a infinite loop I can log any drops in DRAM BW. So sometimes I want to run

taskset -c 0 mbw -n 0 1024 | tee memory.log &

and just let that collect data in the background while I do other things. The taskset is just to ensure the scheduler does not move the process onto another socket when I run on multi-socket servers.


you probably didn't mean to change those to comments in the file


Added this in now thanks. Let me know if there’s anything else i need to do to get it to work?

Appreciate it

Best

Lisa

On 6 May 2015 at 20:02:25, Croydon Dias (notifications@github.com) wrote:

Missing '?showinfo=0&rel=0' at the end.
Adding this prevents related videos from being showed once the video has finished.

—
Reply to this email directly or view it on GitHub.


This commit resolves HospitalRun/frontend#39


Include remote library


Create new instance


Event fired when webcam is ready to stream


nito_test is our room name. make sure this matches on all clients


Video tag to hold the live video of myself


Ok

Посмотрю и отпишусь


a DIV that holds multiple video tags of other participants in the conference


Since lists is a extremely generic feature, isn't it too drastic to add columns like 'toggle_setting_1' and 'toggle_setting_2' for use by a single list which is likely to contain so few entries?


Enabling the script with `chkconfig` appears to cause it located very early in the boot.  This appears to work but I haven't dug to deeply to determine if it's exactly right.

```
S01zfs-import
S02zfs-mount
S30zfs-share
K50zfs-zed
```

I've just started looking over the latest patched so I'll start a new round of review comments there.


why not putting this into RenderFormats.h? iirc it was created to avoid including BaseRenderer in codecs.


Here I manually replace the video tag, cos i only have space for one


pdb usage is probably better removed (in a future cleanup).


NOTE: Though developing branch "sdmcms-navigation" which was created specifically for components related to navigation and navigation management the SdmForm() class was modified for this commit because the navigation manager required checkboxes to be available to it's forms.


good


Tendremos tres dependencias de las librerias jackson para pasar clases javas a json:
jackson-core-2.5.1.jar
jackson-databind-2.5.1.jar
jackson-annotations-2.5.1.jar


This shouldn't ever happen, the compiler won't allow the program to run if amount of parameters is incorrect. See CScriptFunctions::cfclose and all the other compile-time ones.


This is kind of fucked commit probably due to different newlines. It will be hell to work on this file in the future when this commit rewrote the whole file. Can you redo it while keeping the newlines as they are and only change the lines you wanted?


Why not. Currently, I don't really see the need to do that. In any case,  exportScoreParts is only use if convertMode is true. But it could change so better be safe than sorry.


We could still override methods. The constructor was protected.


yes when used only once; but we allow reusing *Frame objects (in loops for example), so the contract is that we modify "retAcc" to be whatever is in the database


This should be an attr_reader, because we don't want classes outside of Barracks to modify the gold and food.


you have it as void defines in header


Thanks Lisa, its live now. The video looks good. :)

On Wed, May 6, 2015 at 8:05 PM, Lisa Altmann-Richer <
notifications@github.com> wrote:

> Added this in now thanks. Let me know if there’s anything else i need to
> do to get it to work?
>
> Appreciate it
>
> Best
>
> Lisa
>
> On 6 May 2015 at 20:02:25, Croydon Dias (notifications@github.com) wrote:
>
> Missing '?showinfo=0&rel=0' at the end.
> Adding this prevents related videos from being showed once the video has
> finished.
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lisaar/gapsummit-site/commit/59574767728189baaa5d46214944886c0f76dd0c#commitcomment-11072541>
> .
>


This method can be refactored in 1 readable line. :)


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/ab8b70e7c15904dfc2057b05e9978d28cb339e99


merging basho/yokozuna/bugfix/zl/aae-expire-and-more = ab8b70e7 into borshop-integration-486-bugfix/zl/aae-expire-and-more


basho/yokozuna/bugfix/zl/aae-expire-and-more = ab8b70e7 merged ok, testing candidate = 100b9242


Sames as can_train_footman. Refactor in 1 line.


@jakebolewski and @jiahao, is this the correct error to throw?


I'd really prefer to see a specific scenario spelled out:

```
// For example, if we have an obligation like `$X = <() as Foo<$X>>::Out`, and trait selection on `() as Foo<$X>` fails, we will unify `$X` with `err`, and expect an error to be reported because `() as Foo<err>` cannot be resolved. But the error is never reported because the predicate contains `err`.
```


thanks for pointing that out


This is pretty dirty, definitely need to think about some refactor :)

Here's a peek at my mongodb now after adding karma in the #general channel in slack:
```
> db.lubot_karma.find({key: "willwh"})
{ "_id" : ObjectId("55020be6147c90d427e12130"), "channel" : "#drupalizeme", "key" : "willwh", "value" : 2496 }
{ "_id" : ObjectId("554a67b091971579b58cb44c"), "channel" : "C024KETLD", "key" : "willwh", "value" : 1 }
```


Same here as on the other commit. Likely newlines killed any sense from this. :(


Good find!


why not just returning CRenderInfo? any reason for this?


But is that in the single user runlevel or the multiuser? Because I haven't tweaked the chkconfig header at all.

For zfs-import it just say chkconfig:    2345 01 99, which should be start at level 1 in runlevel 2 to five. Which isn't single user. Does CentOS even HAVE that (it should, but…)?


Is that loop in make_transfer correct? I don't see any means to exit it, and you're modifying the list in list_for_each_entry. Either I don't understand the code, or it's very wrong.


( you can of course mention the issue # too )


@kevinushey not sure what this line of code is for?  `x$Distance <- x$BP - rep( x$BP[1], nrow(x) )` - BP is for base pair (a numeric/integer), and you're making a vector of length of rows of the data.frame `x`, then subtracting that from the `x$BP` vector - and replacing the `Distance` var - not sure what's going on here - I've commented this line out in a recent commit f2471197d7b7a78b174f341c696cc7a04e44d4d2


This is a mistake. I now wonder if the change in `pu` is a second mistake and someone forgot to declare the variable.


ok


They would inherit but not merge.  If you extend a component that has `positionalParams` specified and you want to use the same params, you do not need to re-declare them.  However, if you want to change the params (or augment them) you would have to declare all of them in the child component.


Awesome :)

Thanks very much

Best,

Lisa
--
Lisa Altmann-Richer
Sent with Airmail

On 6 May 2015 at 20:13:19, Croydon Dias (notifications@github.com) wrote:

Thanks Lisa, its live now. The video looks good. :)

On Wed, May 6, 2015 at 8:05 PM, Lisa Altmann-Richer <
notifications@github.com> wrote:

> Added this in now thanks. Let me know if there’s anything else i need to
> do to get it to work?
>
> Appreciate it
>
> Best
>
> Lisa
>
> On 6 May 2015 at 20:02:25, Croydon Dias (notifications@github.com) wrote:
>
> Missing '?showinfo=0&rel=0' at the end.
> Adding this prevents related videos from being showed once the video has
> finished.
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lisaar/gapsummit-site/commit/59574767728189baaa5d46214944886c0f76dd0c#commitcomment-11072541>
> .
>
—
Reply to this email directly or view it on GitHub.


I'm def. a fan of isXXX methods over `null` checks in the caller code.
And we have those hasXXX methods already so that seems to be reasonable to me.

All in all I think we should raise that as a new issue and complete this PR as it is within itself complete.
As long as the caller knows how many playlists are in the stream he can use it already


heh


This works just fine, but you can refactor this to use `super`. Give it a shot!


Is this part of 4.1.9? A check for this won't really make sense without depending on a release that contains this change, right?


I would suggest removing this paragraph as the actual requirements on implementations are elsewhere. Otherwise you'll have to remember to keep it sync with all the changes, like when I'll shortly suggest that "potentially playing" isn't the best integration point with media elements. (And when I change my mind again and again.)


this is pretty low level, so I would say it is fine


As i also prefer not-code-comments: javadoc for `endOfList()` explaining that it turns `true` as soon as X-ENDLIST was found. Gives at least an idea for a reader in an IDE.


You're using `--privileged` here to bypass SELinux protections from bind mounts presumably?  I understand why, but it's suboptimal.  This is something we need to make better.


You're completely right. Thank you.


Ahh so yeah you are using MinGW on windows for GO.


Totally agree with you. There are some other stuff that likely ahold be moved tout side to a utility class or so.


Yep, I found the same thing, before I checked email again :)


These should be attr_readers. We don't want to generate an attr_writer here.

```ruby
footman = Footman.new
footman.health_points = 100000
```

We wouldn't want to allow code outside of the footman class to increase the footmans health points.


Perhaps you made a typo when you added the include to prelude.cc?

I pushed a commit that adds the include, this builds without issues
here.

On 5 May 2015, at 13:34, jacob-carlborg wrote:

>> Try Bundles → C → Quick Open (⇧⌘D) and enter `cstdlib`.
>
> That finds the header file. This simple example is working fine, so I
> don't think there's something wrong with the SDK:
>
> ```cpp
> #include <cmath>
> #include <cstdlib>
>
> int main (int, char**)
> {
> std::abs(0L);
> return 0;
> }
> ```
>
> If I remove the include for `cstdlib` I'll get a similar compile error
> as above.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/textmate/textmate/commit/4aff7098a5d6e503c6301d2fc49c0b137d754c19#commitcomment-11042667


Works better now, thanks for fixing it so quick. I desided to use your fork because it is ahead of original  one and did not cause issues when I first tried it with Meteor 1.0+, pity that original author has not merged your patches yet (I spotted a pull request).

Many thanks for your efforts to keep this package updated!


This could be done better that just commenting out but we can fix it upstream and whenever you guys want to pull again it will be included.

I was coding for MSVC in Windows and not for MinGW as can be seen by the `#ifdef`.


Hola, tu cambio me parece Adecuado, Gracias


My understanding is that chkconfig will override the passed values in favor of the LSB-style stanza if it exists.  Plus it will honor the listed Requires.  That said, I'm not an expert here but from from the man page.

```
       chkconfig  also supports LSB-style init stanzas, and will apply them in
       preference to "chkconfig:" lines where available.  A LSB  stanza  looks
       like:
       ### BEGIN INIT INFO
       # Provides: foo
       # Required-Start: bar
       # Defalt-Start: 2 3 4 5
       # Default-Stop: 0 1 6
       # Description: Foo init script
       ### END INIT INFO
```

I've also added the following lines to my /etc/fstab for testing and they do get mounted properly on boot.

```
/dev/tank2/ext4		/mnt/ext4		ext4	defaults	0 0
tank1/zfs-filesystem    /mnt/zfs                zfs     defaults        0 0
```

```
$ df -h
Filesystem            Size  Used Avail Use% Mounted on
/dev/mapper/VolGroup-lv_root
                       62G   23G   36G  39% /
tmpfs                 3.9G     0  3.9G   0% /dev/shm
/dev/vda1             477M  117M  335M  26% /boot
/dev/zd0             1008M  1.3M  956M   1% /mnt/ext4
tank1/zfs-filesystem   31G     0   31G   0% /mnt/zfs
```


Leaving this here for now for reference in case it's needed as I add more test cases. When all test cases have been added, I will remove this line if it is still not needed.


`void NORETURN jl_too_many_args(const char *fname, int max)` is probably more consistent


Maybe we should pass in shaders flat?

    options = {
      linkShader: Shader,
      nodeShader: Shader
    }

Then we'll only need to do two checks, but pass in two shaders:

    linkShader && linkShader.vertexCode


This doesn't work anymore so don't even try lolol


Update the comment maximum -> collector


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (348 of 348 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)


Fixed in #1953


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1109


Personally I like striving for the strategy of "make git merge as trivial as possible" with the eye to making future updates simple. Git already has all the logic being applied here to find matching commits, etc. So, generally I wouldn't take out commits and then reapply them. I realize this makes it more difficult to see what changes are outstanding, but there are ways to do it, e.g. `git-cherry`.

Specifically, commits that are cherry-picked back are handled fine by the merge and just show up at a different time in the history. I think that's right since that reflects our actual history. For the changes that should no longer be applied and will actively cause merge conflicts, I would revert them individually if possible prior to the merge.

Having said all that, the approach you've used is perfectly sound and it just comes down to a matter of opinion. Thanks for sharing your reasons for doing it this way.


You could also add the corresponding module in the list I have set, but I
am not sure if it will be valid for all configurations or only ones similar
to yours?
And I would like to understand why this module is required (not really
important though)

On Wednesday, May 6, 2015, Stefan Schlager <notifications@github.com> wrote:

> A simple list(APPEND VTK_LIBRARIES "vtkIOMPIParallel") as an additional
> lib to link against fixes the issues.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/statismo/statismo/commit/2a363b3742dca91c45ff26390eed98447db454ae#commitcomment-11070949>
> .
>


This seems to have broken attachment of containers to OVS bridges causing:

    ovs-vsctl: : missing column name


Actually, I half take it back.  They appear to have mounted properly but only due to a retry.  The following error were reported at boot.  I'll have to dig a little deeper.

```
Remounting root filesystem in read-write mode:             [  OK  ]
Mounting local filesystems:  mount: special device /dev/tank2/ext4 does not exist
filesystem 'tank1/zfs-filesystem' cannot be mounted, unable to open the dataset
                                                           [FAILED]
Enabling /etc/fstab swaps:                                 [  OK  ]
Entering non-interactive startup
```


@d-jsb

Dear Henning
Merlion had a patch for the CRJ700 family that used to be a way to bring this plane to a better state a few years back

I created the branch Merlion, and you may want to  look wether this could be merged into master (?)

Thanks a lot
@IAHM-COL


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (348 of 348 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)


Should this be "Just in Time (JIT) Clients" ?


s/added to the client registry, if/added to the registry if/ (remove the comma)


testing feed


Yup, missed that! replaced react-cookies with local-storage, and added ajax-request to the package.json dependency list


Sincer inca nu am inteles cum sa folosesc yield...


When you say won't detect changes made by user, will it detect on refresh or something?


Oh, I like that :+1:


This is only for the actual hardware, it does not replace the linker for QEMU


...we handle Fridays a little different than the rest of the **week**


can you change this to the correct text?


Better use Map interface. It doesn't limit the user for just HashMap


asdsad


All has been done with help of the .config from them menuconfig and the only thing edit manually on this code was to change "CONFIG_MFD_88PM80x_DVC=y" to "# CONFIG_MFD_88PM80x_DVC is not set"

Also i have renamed the old defconfig from "pxa1088_degaswifi_eur_defconfig" to "pxa1088_degaswifi_eur_defconfig-old". The original one was replaced by the code above witch is the .config (then renamed to pxa1088_degaswifi_eur_defconfig)


Will change.


I don't really know why Tim made this as it is now, but for some reason he seems to have implemented a printf wrapper for debugging for MSVC. I will ask him why but for now the proper solution is not to comment these out.

Instead, since you are using MinGW and not MSVC simply remove compilation of `utils_win32.c` from `CGO`.

The file in question is `ethashc.go` and the line is https://github.com/ethereum/ethash/blob/master/ethashc.go#L12


Since we're on a readability kick... how about if you move the constants from being private members of the class to being local constants within the method? The method *is* the only place you're using them, after all, and it would make it easier to see what's going on without scrolling around.


This approach is also more maintainable, and is more amenable to extending the test suite further down the road.


I think you mean the p? I've changed it to 'sources' and removed some un-used ones


Giulio possibile che a me dia problemi di linking e qualche warning?


@jmchilton @erasche the behavior of booleans seems to be broken. As it seems we need here (in output) `value="--gff3=on"`. `value="True"` does not work.


@Alexandero89 Please always use English comments and do not add this connect anyway ;) Either it's needed or not. So remove the next lines or leave them - depends, so please check and fix.


83811490-Fault_DirtyAirFilter (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1643 of 1643 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)


Making note of the variations we should try to cover.

@willwh: ++
@willwh ++
@willwh:++
@willwh++
and then all those without the @


`line` either has the newline stripped and you have to add one, either ends with a single one and there's nothing to do. See [this](https://docs.python.org/3.4/library/io.html#io.IOBase) to find out which one it is.


Yeah, I think that makes a lot of sense.


Have you "unchkconfig" and "chkconfig" the scripts after each pull test? I've been constantly trying to tweak it and I now think I got it right...


Here, we need "True". Can anyone confirm this with planemo?


I just merged a PR that implement the c code for padding. Can you rebase on top of master to make sure it get tested correctly by travis? (and probably fix a merge conflict)


I'm not really sure what this means here Vasc :(


dsfsdf


fast-forwarding develop to borshop-integration-486-bugfix/zl/aae-expire-and-more = 100b9242


asdasd


Yes! switched to local-storage now with the npm local-storage library. Thanks!


Ironically, a typo in the commit message.


You're on the right track. Great!

Now, you need to be more assertive. Basically, in writing tests you have 100% full control of everything. And you should be explicit about. For example, after you do that `event.save()` you can prove to the code-reader that the event really now is upcoming by inserting an `assert event.is_upcoming()`. And the `if event.is_live() or event.is_upcoming()` doesn't make sense because you should exactly which state it is in.

So no if statement like that is needed. Just do the `ok_('More info' in response.content)` then change the event so it's live and do a new `self.client.get` an run `ok_('More info' in response.content)` again. Then change the event so it's archived and do a new `self.client.get` and now do `ok_('More info' not in response.content)`.


Okay, I see, I changed it

On Wed, May 6, 2015 at 3:35 PM, Kyle Isom <notifications@github.com> wrote:

> This approach is also more maintainable, and is more amenable to extending
> the test suite further down the road.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/timestruck/cfssl/commit/1d285f3a0d41b7f57e2e2f2dcb993d28396230b7#commitcomment-11073122>
> .
>

--
Alice Xia
Cornell University '16


nice! i like the minutes/seconds counter


Doh!


Вместо `API_KEY` вставить публичный ключ. Ну и в `VideosParser.prototype.parse` вернуть `loader` вместо `null`.


That's true, but I think it should have some client-side empty-space handling because it's not very useful and maybe even confusing to have links with no information shown, or no value for the link url


Slow down.  I wanted to eval this with other platforms so we can be consistent.  There are similar needs in iOS, and wp8.


@jeffjewiss yeap, 4.1.9.


Initially we were adding methods to LocalizationUtil, though they were then moved to FullNameDefinitionFactory.java.  This was to remove tight coupling of LocalizationUtil and the FullNameDefinition, so the class could stay generic.  I fear putting this method there would do the same thing.  I'm thinking we simply create a new class (UserNameFieldUtil?).


OK, it does not look like this will work out as I'd hoped on CentOS6.  There's no way that I can see to run `zfs-import` prior to the local mounts.  All the lvm/dm block device code happens in the `rc.sysinit` script prior to pivoting on to the root filesystem.  Bummer.


You seem to have not updated the tests test_DownsampleFactorMaxPaddingStride_grad, test_DownsampleFactorMax_grad, test_DownsampleFactorMax_grad_st, test_DownsampleFactorMax_grad_st_extra, test_DownsampleFactorMaxGrad_grad, test_DownsampleFactorMaxGrad_grad_st, test_DownsampleFactorMaxGrad_grad_st_extra

 to test the grad with sum.

@lamblin @yaoli, do you know where those tests didn't tested the average_* mode of downsample? test_max_pool_2d_2D test the grad of average* (and sum with this PR). Was this enought? I think so, but having confirmation of this would be great.


Thanks for confirming!

On Wed, May 6, 2015 at 3:41 PM Andrey Sitnik <notifications@github.com>
wrote:

> @jeffjewiss <https://github.com/jeffjewiss> yeap, 4.1.9.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/postcss/postcss/commit/d1c465b4baa4224e0e56b126c03b1528b75b632f#commitcomment-11073236>
> .
>


@FernetMenta
Updated https://github.com/popcornmix/xbmc/commit/7aa22258b1300c6f87f729c28c669380ce1028a1


Oops. _Week_ o _days_?


That I"m not sure of. If the function is meant to return a snapshot of a pathway at a given point in time, that specific point in time will have to be provided at some level. Would it be from a user-interaction on a visualization or something?   At what point will a snapshot be requested?


Thanks, @tylerhyperHD!


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (348 of 348 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)


@awanga is this buildable? & if so,has it been tested on actual hardware at all?


Hooooo!!!! Looks like i now need to upgrade my OS to have Java 8, else i cannot compile master. :)


Addresses #1.


@jefffischer That appears to be unintentional - please re-add.


What's `n`?


the same 2 ways exist for submission, as you read in the code.
The chain injection seems to be solely triggered here https://github.com/cms-PdmV/cmsPdmV/blob/v1.8.4/mcm/json_layer/request.py#L615 (there is no button to inject by chain prepid although the API exists) by request prepid. Things work this way only because things are handled in a very specific way that is not meant to be surviving other complex production situations. i.e both ways are needed the way they are implemented, or one (the chain-id injection) can be dropped with further development of the other (the request-id injection).
I see your point on the double locking on request-id, once at ChainRequestInjector instanciation and once at ConfigMakerAndUploader, which sounds to me should never have worked as the configuploader should always fail to upload the config of the root request since the root-request-id is already locked, even with submitting one chain at a time (therefore never reaching thread limits)

But if I read you, it seems the problem is with threads spawning threads: 10 chain injections go in simultaneously, and can never get a new thread for uploading the first config.
Following that argument (which I do not fully understand, see my above confusion), we might be able to get away with it by changing run() with internal_run() at https://github.com/cms-PdmV/cmsPdmV/blob/v1.8.4/mcm/tools/handlers.py#L581
But I think that then, the uploader would skip because the lock on the request-id is already set by the chain injector ; in which case we could get away with it by adding 'inject_'+prepid at https://github.com/cms-PdmV/cmsPdmV/blob/v1.8.4/mcm/json_layer/request.py#L616


There is a maximum value located at DBA1C


Is this a regression - did it use to to use ``--gff3=on``?


@ktretina :clap: Thanks for this script!


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (348 of 348 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)


Cool stuff


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (348 of 348 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)


I only saw the commit so I assumed things. Sorry for that.


Can you tag the release? The Travis build fails because we have a Gemfile.lock in the gem source code which still points to sandthorn-0.7.0. The Gemfile.lock shouldn't even be checked into git when building a gem. To solve this temporarily, always run bundle install before building a gem.

http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/


It matches on any lines with trailing white space after any non-asterisk characters.  A line wrap after multiplication would not match.  Also this is only looking for tabs, spaces, etc at the end of the line.  New lines at end of the line are equivalent to `$` and won't match anyway.


Pasa algo si se ejcuta parse_juzgados antes de que termine parse_ciudades?

La llamada a parse_juzgados no deberia estar en la linea 21?


see forum, even if this mechanism was ok, there would be an issue with this. you change behaviour by returning true which signals that the event was taken. Listeners down the chain won't see it anymore.


OK then, missed that


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (348 of 348 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)


I propose that rather than using getHeaderList and getSplittableHeader, we use getHeaderList and getHeaderLines. The names can be changed, of course, but the idea is to let the default getHeaderList method return a list of individual header values while the getHeaderLines method would return a list of header line values (i.e., opaque header values that may be multiple comma-delimited values). The same could be done for getPreferredHeader and getPreferredSplittableHeader.

Also, we have signatures to make adding quality easier, but not any other parameter. Perhaps we could generalize that?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


Why this change ?
"systemjs" does not reflect the name of the project on bower, it is related to mrdoob/systemjs resulting in wrong package installation.


not in the slightest, unfortunately. I'm basing it on the 3.10 minnow source, which is theoretically functional, but there have been alot of changes to the ARM/OMAP portion of linux since 3.0, so at this point it is just porting the board files.


Are you sure there is a namespace separator?  If not what will the behavior be here?


You got it!


pardon my ignorance, but why have a blocksize limit at all?
Shouldn't the block size be a function of the transaction volume anyway?
If for some awesome reason Bitcoin transaction volume all of a sudden explodes, won't block size limits simply be in the way? and if there's no transaction volume growth you will still use less than the size you happen to be debating on.


ok, its a good way.


okay.  the generic name of the function doesn't make it clear it's only used there, but it seems to be so whatever fits this use case is fine.  BTW the function `ui_menu_sort_by_label()` should probably be made static, it doesn't seem to be used anywhere but in ui_utils() itself (it is in the header, but no code seem to use it)


Is the local variable "basename" necessary?


I think this used to work at some point. But even if not this is inconsistent. But hey I should get sleep and retest everything tomorrow.


I would use the $timeout service here.

```
$timeout(function() {
dataLayer.push({
 			'event': 'content-view',
 			'content-name': path
 		});
});
```

That way it allows Protractor and others to be notified when things get added to the stack. Otherwise it can't notify anything with setTimeout as it's a native construct


that dal commit doesn't exist .... typo ?!


Could base ever be null?


Is this just to be explicit about our intentions?


Just a nitpick, you can do whatever but this can be simplified to just:

`constructor: (@url, @user) ->`

(http://coffeescript.org/#classes)


no need to initialise a vector with clear(). or did I miss something?


r+


That makes no sense.


Actually no.


Thanks for your comments, Dan. To be clear, I do agree with you in the general case (which is what I did in other occasions) but here, in this particular situation, I do think it's a bit cleaner and easier to handle if I do it this way, as git-cherry would not be very clear after the merge, due to the conflicts.

Hopefully, next time we upgrade PA the merge will be clean right from the start and I won't feel the urge to do this hackish workaround again. Thanks for your understanding!


@TuckerJD I should have been more specific - these "undefined" errors were showing up for me in the FirefoxDevelopers javascript console.  Maybe it was a problem with be debug setup.


Any chance of "attributes" being null ?


I'm sure you're right - will remove.


looks goods apart from the minor


Thanks, fixed it!


This still does not clearly define what to set this to or how to use it.


parse_juzgados necesita la información de que municipios existen para sacar los juzgados de cada municipio
`"http://www.poderjudicial-gto.gob.mx/includes/oficinas.php?municipio=#{municipio}"`


I believe it - there was a very similar regression with ``select`` params. https://github.com/galaxyproject/galaxy/commit/5b6c092aa0a85455b4daacd318541e92bd08b876


The build-tools dependency sounds fine to me.  However, If we go down the route for the java-formatter, we should also change (in a follow-on issue) other build tools (checkstyle, findbugs, etc) to use the build-tools module for configuration.  Does anyone (especially those who have worked on findbugs/checkstyle) have any objections to a build-tools module?


maybe yeah.  not sure how important is is tho, but might be a good idea.


Removed the `declare` statements entirely in fc290fbd859a0ffab1d11a061a6e56e15ddc1fde.


I gonna merge it, but a few points before:
- Check the code style
- Second `if` replace with `else if`
- `===` instead of `==`


I think "flag" (or "switch") is a standard name for short options (with just one character and single `-`). I'd keep "flag".


Good catch!!!


Why i always accept this page's e-mail?? What the fuck。。。。


Does anyone understand why this might have been failing? @vtjnash, @Keno, @JeffBezanson?


Look at the updated code
On May 5, 2015 6:57 PM, "goldfish" <notifications@github.com> wrote:

> i was trying to make it so there wouldn't be as much work for you. you
> could actually not have winOrLoss at all, if you want. if you just update
> totalDealer and totalPlayer points, i can determine win or loss within
> Wager. less work for you and it'd keep them separate which would help
> streamline the code
>
> On Tue, May 5, 2015 at 6:35 PM, Zubz21 <notifications@github.com> wrote:
>
> > I could take care of that scenario but it would be downgrading the task
> for
> > your wager. After taking care of the victor i will determine your
> > winorloss. So you will only have to pass in the winorloss and the bet
> > instead. Which makes the wager a whole lot easier.
> >
> > This would be more work for me but i feel like it would look cleaner
> >
> > On May 5, 2015 6:28 PM, "Zubeir Said" <said.zubeir@gmail.com> wrote:
> >
> > > What do you think is best. Leaving it in main thw way it is or putting
> it
> > > into a function. I think if we use a function we will have to return
> too
> > > many things
> > > On May 5, 2015 6:14 PM, "goldfish" <notifications@github.com> wrote:
> > >
> > >> i'm not sure, but i think the play function arguments will at least
> need
> > >> (char& winOrLoss, int& totalPlayer, int& totalDealer) for arguments,
> > >> because i need them for the wager function. if you have another way
> > you'd
> > >> prefer, i can adjust wager. right now i tried to calculate different
> > >> win/loss scenarios in Wager , so that you wouldnt need to have a bunch
> > of
> > >> loops checking that stuff in your play function.
> > >>
> > >> —
> > >> Reply to this email directly or view it on GitHub
> > >> <
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057510
> > >
> > >> .
> > >>
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057634
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057852>
> .
>


- Added another great reason to use usbkill!


Agreed!


rax retry tests


Either or, but not "machine/computer". I prefer "machine", but as I said it's personal. Also, whatever you decide, make it consistent with the similar change you have in the other file.


Woops I didn't even realize that behaviour had different american/british spellings. I did think that I might have misspelled it and double checked, but I guess my mental comparison functioned word-wise and it saw that all words were spelled correctly. Maybe it could be named `auto-completion-return-key-mode`.


okay i can update wager to have a case for ans = 'P'. will that work?


why delete this? The purpose is to prevent people from mistyping @ directives in tests and not noticing


I don't know if this is the cause of the issue you've reported, but the second argument to the POSIX call is the length, not the offset.  You're passing a path, not a file descriptor, so there's no notion of position in the file.


I thought we want to magic up those dependencies?


Thanks. I'm still finding my way around while getting a wiki up and running at work. I'll try to organize our changes in the next few weeks and make some plugins and/or patches.

> On May 5, 2015, at 20:48, Martín Gaitán <notifications@github.com> wrote:
>
> Cool idea. Consider to publish it as a custom app/plugin.
>
> —
> Reply to this email directly or view it on GitHub.
>


I think we can remove the "(like this one)" given that we are using this mostly online.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.79](http://teamcity.monogame.net/viewLog.html?buildId=6989&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 485, ignored: 6 Build time: 00:33:49


Why use mcpelauncher.h when you have Substrate.h?


distributionUrl=http\://services.gradle.org/distributions/gradle-2.3-bin.zip, not all, we do not want source.s


Good to go.


Testing per-line-commenting.


Fixed in ee3731ccd1e111214632b075cdabd5d888408e63.


I think here you want to use RIPEMD-160(SHA-256(asset definition))


what is multilevel? oh, and you use VERY complicated methods to create a block, with the ones I don't want to deal with, honestly.


Ya lo he echado a andar. Gracias!   La opción sandbox es solo para pruebas porque da algún error?


Yes that will work. Do you think could test the program and figure out why
it doesnt update the wallet. Maybe it needs to return it? Also in regards
to the Ace. it is set as a and not as an ace so i cant switch it to be 1 or
11. I believe that's David's part though, correct?

On Tue, May 5, 2015 at 7:09 PM, goldfish <notifications@github.com> wrote:

> okay i can update wager to have a case for ans = 'P'. will that work?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057938>
> .
>


This is wikimedia format, so * doesn't do anything. Use single quotes for bold or italic.


What about simplifying a bit, something along these lines: "Not only teams, but also lone researchers can benefit from using a version control system: keeping a record of what was changed, when, and why is very useful if the researcher needs to come back to the project later on"


There we go @jimaek


Also, I'm not sure if we should be trying to decode the filename.  The POSIX call just works on a `char*`.  But I've never worked with C strings that weren't `wchar` or some similar wrapper, so I might have that wrong.


Can you [capitalize the commit message and use imperative mode](http://chris.beams.io/posts/git-commit/)?
i.e.:
Introduce compact pure hash form of the asset definition pointer


You could've use `GenericMath.lerp(entity.prevPosX, entity.posX, partialTicks);`


Still a few places where Rubocop was referenced, eg: https://github.com/brigade/overcommit/blob/68a000f1e515ccabd4c7e9c69514bcc245054bf2/config/starter.yml#L19, which makes the default config not working


Utilizamos o List porque queremos obter todos os Eventos que o Estabelecimento tem. Tem que importar o List porque ela não esta no pacote "java.lang", as que estão no pacote "java.lang" são importadas implicitamente. Fique a vontade para perguntar ;)


Same story for me, unicode characters get stripped. Now I'm looking at messages like this.
![image](https://cloud.githubusercontent.com/assets/350654/7486652/965228ba-f3eb-11e4-8abc-8fedd540890c.png)


This broke POST requests.


can you change 'wait' to 'sleep'? It's a bit clearer that your program will block then.


Typo? MAP_FROM_KERNEL_CORE rather than MAP_FORM_KERNEL_TO_CORE


I build with `-Weverything`, which gives the error when analyzing.


yes, shuffle and displaying the remaining cards are david. i think he needs
to change the way the cards are identified to strings. i've fixed the
wallet not displaying, am gonna upload after i test it a little more.

On Tue, May 5, 2015 at 7:22 PM, Zubz21 <notifications@github.com> wrote:

> Yes that will work. Do you think could test the program and figure out why
> it doesnt update the wallet. Maybe it needs to return it? Also in regards
> to the Ace. it is set as a and not as an ace so i cant switch it to be 1 or
> 11. I believe that's David's part though, correct?
>
> On Tue, May 5, 2015 at 7:09 PM, goldfish <notifications@github.com> wrote:
>
> > okay i can update wager to have a case for ans = 'P'. will that work?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057938
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058051>
> .
>


Same as above


LLVM 3.3 I presume?


thank's for sharing


Pyro you are amazing with coding.


ah ok I missed that .... thanks!


A atributo identificador está como BigInteger, então seria <BigInteger, AlunoCurso>. Acho melhor passar tudo para Long, tanto ao Extender o DAO quando na classe.


Yep.


otamesi


So what bower package did you install for jquery-ui? The one I used is jquery.ui and only have the main file not the sub files of sortable or draggable.


Ok well im not sure what to work on now. Im sure there  is a lot. What
would you like me to do
On May 5, 2015 7:33 PM, "goldfish" <notifications@github.com> wrote:

> yes, shuffle and displaying the remaining cards are david. i think he needs
> to change the way the cards are identified to strings. i've fixed the
> wallet not displaying, am gonna upload after i test it a little more.
>
> On Tue, May 5, 2015 at 7:22 PM, Zubz21 <notifications@github.com> wrote:
>
> > Yes that will work. Do you think could test the program and figure out
> why
> > it doesnt update the wallet. Maybe it needs to return it? Also in regards
> > to the Ace. it is set as a and not as an ace so i cant switch it to be 1
> or
> > 11. I believe that's David's part though, correct?
> >
> > On Tue, May 5, 2015 at 7:09 PM, goldfish <notifications@github.com>
> wrote:
> >
> > > okay i can update wager to have a case for ans = 'P'. will that work?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057938
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058051
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058131>
> .
>


maybe contact david and see if he needs help? or maybe start on our
enhancement? what do you want to do for the extra thing?

On Tue, May 5, 2015 at 7:37 PM, Zubz21 <notifications@github.com> wrote:

> Ok well im not sure what to work on now. Im sure there is a lot. What
> would you like me to do
>
> On May 5, 2015 7:33 PM, "goldfish" <notifications@github.com> wrote:
>
> > yes, shuffle and displaying the remaining cards are david. i think he
> needs
> > to change the way the cards are identified to strings. i've fixed the
> > wallet not displaying, am gonna upload after i test it a little more.
> >
> > On Tue, May 5, 2015 at 7:22 PM, Zubz21 <notifications@github.com> wrote:
> >
> > > Yes that will work. Do you think could test the program and figure out
> > why
> > > it doesnt update the wallet. Maybe it needs to return it? Also in
> regards
> > > to the Ace. it is set as a and not as an ace so i cant switch it to be
> 1
> > or
> > > 11. I believe that's David's part though, correct?
> > >
> > > On Tue, May 5, 2015 at 7:09 PM, goldfish <notifications@github.com>
> > wrote:
> > >
> > > > okay i can update wager to have a case for ans = 'P'. will that work?
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057938
> > > >
> > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058051
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058131
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058159>
> .
>


nice one :+1:


@tmat I was afraid that the behavior of step would change. For example, non-constructor method would stay twice at '{' in the presence of closure:

public void Foo()
{
   ...
}

It looks like that NeedSequecePoints should return true for constructors as in your example.


文字化けしてますね。メモファイルは、テキストエディタ（桜エディタ）で作成したので、シフトＪＩＳコードだからですね。


Should be 0, 193, 222


@secrettriangle, it looks like iTerm2 officially moved to github:
https://github.com/gnachman/iTerm2

But issues are tracked on the gitlab repo: https://iterm2.com/bugs https://gitlab.com/gnachman/iterm2/issues/new?issue


@nirvdrum yes I think I need to just rename offset, it's supposed to be length, not certain on the decoding either


There will not be any issues, but i feel it not good practise.
sai.h file contains only api query and initialize functions. Process where api query needs this will include this. Example i am writing the ACL application than only 2 or 3 then i need api_query for method table and saiacl.h, saitypes.h, saistatus.h is enough. Why we need to include all other sai*.h files to my ACL app.


It's right


oh! play needs to be in a function. in the version i see, it is still in
main...

On Tue, May 5, 2015 at 7:39 PM, miss goldfish <fishcaneatpeople@gmail.com>
wrote:

> maybe contact david and see if he needs help? or maybe start on our
> enhancement? what do you want to do for the extra thing?
>
> On Tue, May 5, 2015 at 7:37 PM, Zubz21 <notifications@github.com> wrote:
>
>> Ok well im not sure what to work on now. Im sure there is a lot. What
>> would you like me to do
>>
>> On May 5, 2015 7:33 PM, "goldfish" <notifications@github.com> wrote:
>>
>> > yes, shuffle and displaying the remaining cards are david. i think he
>> needs
>> > to change the way the cards are identified to strings. i've fixed the
>> > wallet not displaying, am gonna upload after i test it a little more.
>> >
>> > On Tue, May 5, 2015 at 7:22 PM, Zubz21 <notifications@github.com>
>> wrote:
>> >
>> > > Yes that will work. Do you think could test the program and figure out
>> > why
>> > > it doesnt update the wallet. Maybe it needs to return it? Also in
>> regards
>> > > to the Ace. it is set as a and not as an ace so i cant switch it to
>> be 1
>> > or
>> > > 11. I believe that's David's part though, correct?
>> > >
>> > > On Tue, May 5, 2015 at 7:09 PM, goldfish <notifications@github.com>
>> > wrote:
>> > >
>> > > > okay i can update wager to have a case for ans = 'P'. will that
>> work?
>> > > >
>> > > > —
>> > > > Reply to this email directly or view it on GitHub
>> > > > <
>> > >
>> >
>> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057938
>> > > >
>> > >
>> > > > .
>> > > >
>> > >
>> > > —
>> > > Reply to this email directly or view it on GitHub
>> > > <
>> >
>> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058051
>> > >
>> > > .
>> > >
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058131
>> >
>>
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058159>
>> .
>>
>
>


The enhancement. I was thinking maybe about save game?
On May 5, 2015 7:40 PM, "goldfish" <notifications@github.com> wrote:

> maybe contact david and see if he needs help? or maybe start on our
> enhancement? what do you want to do for the extra thing?
>
> On Tue, May 5, 2015 at 7:37 PM, Zubz21 <notifications@github.com> wrote:
>
> > Ok well im not sure what to work on now. Im sure there is a lot. What
> > would you like me to do
> >
> > On May 5, 2015 7:33 PM, "goldfish" <notifications@github.com> wrote:
> >
> > > yes, shuffle and displaying the remaining cards are david. i think he
> > needs
> > > to change the way the cards are identified to strings. i've fixed the
> > > wallet not displaying, am gonna upload after i test it a little more.
> > >
> > > On Tue, May 5, 2015 at 7:22 PM, Zubz21 <notifications@github.com>
> wrote:
> > >
> > > > Yes that will work. Do you think could test the program and figure
> out
> > > why
> > > > it doesnt update the wallet. Maybe it needs to return it? Also in
> > regards
> > > > to the Ace. it is set as a and not as an ace so i cant switch it to
> be
> > 1
> > > or
> > > > 11. I believe that's David's part though, correct?
> > > >
> > > > On Tue, May 5, 2015 at 7:09 PM, goldfish <notifications@github.com>
> > > wrote:
> > > >
> > > > > okay i can update wager to have a case for ans = 'P'. will that
> work?
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057938
> > > > >
> > > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058051
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058131
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058159
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058168>
> .
>


Green should be 119, 175, 65


I think we should fix it as follows:

```C#
private BoundBlock RewriteBlock(BoundBlock node, ArrayBuilder<BoundExpression> prologue, ArrayBuilder<LocalSymbol> newLocals)
        {
            RewriteLocals(node.Locals, newLocals);

            var newStatements = ArrayBuilder<BoundStatement>.GetInstance();

            foreach (var statement in node.Statements)
            {
                var replacement = (BoundStatement)this.Visit(statement);
                if (replacement != null)
                {
                    if (newStatements.Count == 0 && prologue.Count > 0)
                    {
                        for (int i = 0; i < prologue.Count; i++)
                        {
                            BoundStatement prologueStatement = new BoundExpressionStatement(prologue[i].Syntax, prologue[i]);

                            // transfer the bound sequence point from the first rewritten statement to the first prologue expression statement:
                            if (i == 0)
                            {
                                BoundSequencePoint sequencePoint;
                                BoundSequencePointWithSpan sequencePointWithSpan;
                                if ((sequencePointWithSpan = replacement as BoundSequencePointWithSpan) != null)
                                {
                                    prologueStatement = new BoundSequencePointWithSpan(sequencePointWithSpan.Syntax, prologueStatement, sequencePointWithSpan.Span);
                                    replacement = sequencePointWithSpan.StatementOpt;
                                }
                                else if ((sequencePoint = replacement as BoundSequencePoint) != null)
                                {
                                    prologueStatement = new BoundSequencePoint(sequencePoint.Syntax, prologueStatement);
                                    replacement = sequencePoint.StatementOpt;
                                }
                            }

                            newStatements.Add(prologueStatement);
                        }

                        if (replacement == null)
                        {
                            continue;
                        }
                    }

                    newStatements.Add(replacement);
                }
            }

            prologue.Free();

            // TODO: we may not need to update if there was nothing to rewrite.
            return node.Update(newLocals.ToImmutableAndFree(), newStatements.ToImmutableAndFree());
        }
```


Ah, nevermind. Looks like the old issues are still there.

https://gitlab.com/gnachman/iterm2/issues/3239
https://gitlab.com/gnachman/iterm2/issues/3052
https://gitlab.com/gnachman/iterm2/issues/2936


Maybe add offline installer download links?


Technically ioc_exits check is sufficient as we dont care what ISA it is for - it will only be set for ISA v2.
But that means for ARCompact we do a useless check. So we need both checks after all.
But rather than propagate this everywhere, lets make ioc_exists() a static inline which does that - OK


fixed! o(╯□╰)o


we would save the data into a file and then it would read from the file at
the start of a new game? we could do that. i think we need play to be a
function, first, though.

On Tue, May 5, 2015 at 7:44 PM, Zubz21 <notifications@github.com> wrote:

> The enhancement. I was thinking maybe about save game?
>
> On May 5, 2015 7:40 PM, "goldfish" <notifications@github.com> wrote:
>
> > maybe contact david and see if he needs help? or maybe start on our
> > enhancement? what do you want to do for the extra thing?
> >
> > On Tue, May 5, 2015 at 7:37 PM, Zubz21 <notifications@github.com> wrote:
> >
> > > Ok well im not sure what to work on now. Im sure there is a lot. What
> > > would you like me to do
> > >
> > > On May 5, 2015 7:33 PM, "goldfish" <notifications@github.com> wrote:
> > >
> > > > yes, shuffle and displaying the remaining cards are david. i think he
> > > needs
> > > > to change the way the cards are identified to strings. i've fixed the
> > > > wallet not displaying, am gonna upload after i test it a little more.
> > > >
> > > > On Tue, May 5, 2015 at 7:22 PM, Zubz21 <notifications@github.com>
> > wrote:
> > > >
> > > > > Yes that will work. Do you think could test the program and figure
> > out
> > > > why
> > > > > it doesnt update the wallet. Maybe it needs to return it? Also in
> > > regards
> > > > > to the Ace. it is set as a and not as an ace so i cant switch it to
> > be
> > > 1
> > > > or
> > > > > 11. I believe that's David's part though, correct?
> > > > >
> > > > > On Tue, May 5, 2015 at 7:09 PM, goldfish <notifications@github.com
> >
> > > > wrote:
> > > > >
> > > > > > okay i can update wager to have a case for ans = 'P'. will that
> > work?
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub
> > > > > > <
> > > > >
> > > >
> > >
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057938
> > > > > >
> > > > >
> > > > > > .
> > > > > >
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058051
> > > > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058131
> > > >
> > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058159
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058168
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058201>
> .
>


also i just found a bug, i think in play, let me upload screencap

On Tue, May 5, 2015 at 7:46 PM, miss goldfish <fishcaneatpeople@gmail.com>
wrote:

> we would save the data into a file and then it would read from the file at
> the start of a new game? we could do that. i think we need play to be a
> function, first, though.
>
> On Tue, May 5, 2015 at 7:44 PM, Zubz21 <notifications@github.com> wrote:
>
>> The enhancement. I was thinking maybe about save game?
>>
>> On May 5, 2015 7:40 PM, "goldfish" <notifications@github.com> wrote:
>>
>> > maybe contact david and see if he needs help? or maybe start on our
>> > enhancement? what do you want to do for the extra thing?
>> >
>> > On Tue, May 5, 2015 at 7:37 PM, Zubz21 <notifications@github.com>
>> wrote:
>> >
>> > > Ok well im not sure what to work on now. Im sure there is a lot. What
>> > > would you like me to do
>> > >
>> > > On May 5, 2015 7:33 PM, "goldfish" <notifications@github.com> wrote:
>> > >
>> > > > yes, shuffle and displaying the remaining cards are david. i think
>> he
>> > > needs
>> > > > to change the way the cards are identified to strings. i've fixed
>> the
>> > > > wallet not displaying, am gonna upload after i test it a little
>> more.
>> > > >
>> > > > On Tue, May 5, 2015 at 7:22 PM, Zubz21 <notifications@github.com>
>> > wrote:
>> > > >
>> > > > > Yes that will work. Do you think could test the program and figure
>> > out
>> > > > why
>> > > > > it doesnt update the wallet. Maybe it needs to return it? Also in
>> > > regards
>> > > > > to the Ace. it is set as a and not as an ace so i cant switch it
>> to
>> > be
>> > > 1
>> > > > or
>> > > > > 11. I believe that's David's part though, correct?
>> > > > >
>> > > > > On Tue, May 5, 2015 at 7:09 PM, goldfish <
>> notifications@github.com>
>> > > > wrote:
>> > > > >
>> > > > > > okay i can update wager to have a case for ans = 'P'. will that
>> > work?
>> > > > > >
>> > > > > > —
>> > > > > > Reply to this email directly or view it on GitHub
>> > > > > > <
>> > > > >
>> > > >
>> > >
>> >
>> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057938
>> > > > > >
>> > > > >
>> > > > > > .
>> > > > > >
>> > > > >
>> > > > > —
>> > > > > Reply to this email directly or view it on GitHub
>> > > > > <
>> > > >
>> > >
>> >
>> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058051
>> > > > >
>> > > > > .
>> > > > >
>> > > >
>> > > > —
>> > > > Reply to this email directly or view it on GitHub
>> > > > <
>> > >
>> >
>> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058131
>> > > >
>> > >
>> > > > .
>> > > >
>> > >
>> > > —
>> > > Reply to this email directly or view it on GitHub
>> > > <
>> >
>> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058159
>> > >
>> > > .
>> > >
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058168
>> >
>>
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11058201>
>> .
>>
>
>


Doesn't this mean we mutate a string constant on line 206?


Very good thinking on your part - just the implementation needs to be tweaked a bit to make it look nicer !
i took the same approach for dcache aliasing check !


Also, you didn't remove the `free` on line 212.


@tmat Great! I will try this approach. Thanks!


@majianpeng this change could introduce extra overhead because in this change, we call `_crc_update_clone_range()` for each copied extent. and this call in turn touches the disk each time it gets called. see `chain_fgetxattr()`. so probably in addition to using `_do_copy_range()`, we might want to extract the call to `_crc_update_clone_range()` out of `_do_copy_range()`. what do you think?


So great! In travis, some tests have been failed yet. It has been confusing me. I'm very pleased you've fixed it! Thanks a lot!


This is definitely an improvement for readability.  There's an obvious problem with equivalent (present in the original PR also) which is that two unsized arrays with different initializer lengths will return false.


Oops. Didn't notice it updated itself when I launched the program and that it committed.


好像发现耗电无比。。。revert掉试试。。


Just some stuff.. Let's just get rid of this/


if  enableTransactionSupport = true and TransactionSynchronizationManager.isActualTransactionActive()==false, can not close RedisConnection


Desmume (windows) support action replay cheats (AR)

1). File -> Open Rom... -> (Pick your ROM)
2). Emulation -> Cheats -> List -> ["Cheats list" window]
3). Click the Action Replay button -> ["Add Action Replay code" window]
4). Paste your AR code and give it a description, then click Add button
5). Repeat Steps 3~4 for each code you want
6). Here's the tricky part... on the "Cheats list" window, press the SAVE button

Also support usrcheat.dat take a loock here https://www.youtube.com/watch?v=C5SiFvVWS7s


kk will revert


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


saw approval from MonsieurNicolas
at https://github.com/jedmccaleb/stellar-core/commit/df2f4bb1d8df772d6aabc790be8270e7eb91e143


merging jedmccaleb/stellar-core/master = df2f4bb1 into auto


jedmccaleb/stellar-core/master = df2f4bb1 merged ok, testing candidate = 795da685


Hm yeah, by us. Actually pretty much only by me so far since I just established that meaning. Suggestions?


What does this mean? I didn't have to type vagrant when I used sudo?


Let's not give up entirely on a `zfs_action` function.  We may not be able to use it everywhere, but there are clearly places where it would be useful like in `zfs-mount`.  For example.

Added to common.init.in
```
zfs_action()
{
        local MSG rc

        MSG=$1
        zfs_log_begin_msg "$MSG "
        shift
        "$@"
        ret=$?
        if [ $ret -eq 0 ]; then
                zfs_log_end_msg $"$MSG"
        else
                zfs_log_failure_msg $"$MSG"
        fi

        return $ret
}
```

And for example in `zfs-mount`

```
                zfs_action $"Mounting ZFS filesystem(s)" \
                    $ZFS mount -a$verbose$overlay $MOUNT_EXTRA_OPTIONS
```


Yeah, I added this parameter to all `_line_generator` calls just to be explicit. I think it helps with readability a little.


This seems to fix the proxy crashing.

The latest checkout from github doesn't compile, however taking the latest stable (2.3.2) and applying these changes manually works and fixes the problem.

The proxies still go from peers being "???" to #'s, and sometimes back to "???" (not sure what causes this - possibly overloaded? -- this is on a 20Gb link with 4 proxies), but it no longer crashes.

Thanks for fixing!


wtf man


Let's carve it up like this:

* Every state in every rollout has a unique identifier that allows for that exact state to be reconstructed using simulation from the starting state of the pathway.
* The visualization renders the rollouts and eventually the user may decide they want to view more details about a particular state in a particular rollout.
* The visualization then requests the variables of the state that were not returned in the initial request. The way the server knows what state to return is by processing the unique identifier that is included in the rollout. The additional variables returned in this request are likely the complete landscape object since it was not given to the visualization in the first request.

So I _think_ you need to add an identifier to the rollouts for every state. Perhaps {eventNumber, pathwayNumber} in addition to all the policy parameters that made the rollout result in that state.


@luissifu una forma que estaria mejor para esto (creo yo) es algo asi
```ruby
def initialize(f = Date.today -1)
@fecha = f
end
```
lo que hace esto es que si no viene defenido con fecha le aplica el dia de ayer pero si si viene definido toma ese que le estas mandando


Yes.

I feel like this paragraph doesn't stand out well enough however but I don't want to clutter the page with many .. warning boxes.


Thanks.


why is this the only class that's documented?


Might be overkill but mentioning vagrant ssh and logout might be good.


Should we use a real list?


do not use linux ..
tried to compile?
give error?
Enable large pages in the operating system?

2015-05-06 17:20 GMT+02:00 snino64 <notifications@github.com>:

> I can not compile deepfishz in linux.
> I made some corrections and now if I can compile.
> Are changes correct?
>
> /*
> Stockfish, a UCI chess playing engine derived from Glaurung 2.1
> Copyright (C) 2004-2008 Tord Romstad (Glaurung author)
> Copyright (C) 2008-2015 Marco Costalba, Joona Kiiski, Tord Romstad
>
> Stockfish is free software: you can redistribute it and/or modify
> it under the terms of the GNU General Public License as published by
> the Free Software Foundation, either version 3 of the License, or
> (at your option) any later version.
>
> Stockfish is distributed in the hope that it will be useful,
> but WITHOUT ANY WARRANTY; without even the implied warranty of
> MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> GNU General Public License for more details.
>
> You should have received a copy of the GNU General Public License
> along with this program. If not, see http://www.gnu.org/licenses/.
> */
> #include
> #include //<-----
>
> #include "thread.h"
> #include "tt.h"
>
> using namespace std;
>
> #define TRUE 1
> #define FALSE 0
>
> int large_use;
>
> #ifndef _WIN32 // Linux
> #include
> #include
>
> static int num;
>
> #define MEMALIGN(a, b, c) posix_memalign ((void**) &(a), b, c) //<--------
> #define ALIGNED_FREE(x) free (x); //<--------
>
> void SETUP_PRIVILEGES(){} //<--------
>
> void CREATE_MEM(void** A,int align,uint64_t size)
> {
> large_use=FALSE;
>
> num=shmget(IPC_PRIVATE,size,IPC_CREAT|SHM_R|SHM_W|SHM_HUGETLB);
> if(num==-1)
> {
>
> // printf("info string LargePages FAILED %llu Mb\n",size>>20);
> std::cout << "info string LargePages FAILED %llu Mb\n" << (size >> 20) <<
> std::endl;
> MEMALIGN((
>
>
>
> *A),align,size); } else { (*A)=shmat(num,NULL,0x0);
> large_use=TRUE;
> // printf("info string LargePages OK %llu Mb\n",size>>20);
> std::cout << "info string LargePages OK %llu Mb\n" << (size >> 20) <<
> std::endl;
> std::cout<<"info string HUGELTB "<<(size>>20)<<std::endl;
> }
> }
>
> void CREATE_MEM2(void** A,uint64_t size)
> {
> large_use=FALSE;
> num=shmget(IPC_PRIVATE,size,IPC_CREAT|SHM_R|SHM_W|SHM_HUGETLB);
> if(num!=-1)
> {
> (*A)=shmat(num,NULL,0x0);
> large_use=TRUE;
> // printf("info string %llu Mb LargePages\n",size>>20);
> std::cout << "info string %llu Mb LargePages\n" << (size >> 20) <<
> std::endl;
> }
> }
>
> void FREE_MEM(void* A)
> {
> if(!A)
> return;
> if(!large_use)
> {
> ALIGNED_FREE(A);
> return;
> }
> shmdt(A);
> shmctl(num,IPC_RMID,NULL);
> large_use=FALSE;
> }
>
> #else
>
> Thanks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Zerbinati/DeepFishMZ/commit/30f2afc3e63eb03b2a5c2a30b972fa55372f2f71#commitcomment-11067845>
> .
>


Just another nit:

You seem to be calling `if @validChannel msg` at the beginning of all of the functions you call from this switch statement. It might make sense to make a separate function that performs this check on `msg` before calling the proper function instead. You could maybe do this before the `switch` or by wrapping the functions in some check.

For example:

```when "nominate" then checkMessage msg, aotw.nominate```

Where `checkMessage` is something like this:

```
checkMessage = (msg, command) ->
  if @validChannel msg
    command msg
```

Just a thought, it might make it a little cleaner than the extra indentation level in each of those processing commands.


OK, given I have already pushed this I will add a correction via a new commit which will get added to this pull request.


This example took me a long time just to download the image. Might want to just demo this one.


`nb`? Serios?


Eeeek massive commit @craigacronin it's a little hard for me to see what was changed & review. Would it be possible to do a new PR with more granular commits please instead?


I'll reword that. Basically, it doesn't make sense to ignore packages that don't exist, so we might as well clean that up.


@keertip
We must put lower/upper bounds on these. There are breaks in unittest, for example.


I agree, this message might be confusion to new users. But IMHO it is more of a wording issue than the lack of additional description. I guess this originates in the of the 'access key'/'Zugangsschlüssel'. Maybe we should replace it by (session) ID which is used in other places.


Good stuff! Glad to see someone else using CoffeeScript around here :smiley:


Ăsta nu-i același cod din constructor?


We should consider `balance leastconn` here as the selection algorithm since SSH connections are longer lived than http.


oh, I didn't see those as they overflow the diff.  but I really don't mind, I just thought it would have been more concise, but I also don't know anything about Waf so…


Tal vez _days_ sea más preciso


No, we can't be sure of that, if false, the substring call will return -1, the +1 gets us back to the start of the string.
This logic could move up to the first test, but I did not want the best case scenario (hashtable hit) to suffer due to this stringwise operation.


@PamBWillenz - I think the assignment is supposed to have a Question model.


commit the sync from VS


LGTM


People will go through the pre-requisites defined in the top level readme prior to the demo. One pre-req step is to download the box so that the first vagrant up won't take a long time.


That is by intention. As long as airtunes is streamed this is what i want...


Travaux effectués au Cégep entre 2012 et 2014


Is it bad that we possibly call this twice? Should we set a flag so we only call it once?


You can also import "modules" in the package root so "submodules" is not exactly correct.


Good point. I'll add that.


also i could even return false here - doesn't really matter for airtunes - as long as i get the notification about someone pressed the >>| button...


It's like you add a file `Packages/Default/myexec.py` and write `from . import exec` within.


If you feel this docs could be improved, please send us a PR


Initial commit!


I'm not sure about punctuation in english sentences (as to where it's optional, required or wrong), but I think this one is more on the wrong side.


as colt mentioned, I guess we'd want a moderator or admin of the channel to be set in the constructor, and then in nominate, we'd check to see if the person nominating the album is the moderator


Замени еще "на этом" на "в этом уроке", так будет лучше звучать и читаться


are there any encoding that has letters in it?  e.g. I think a dash ('-') is %2d or %2D


What is that red encoding around the 2nd dataset (NEPAnode MapWarper)? Are there embedded ASCII escape codes?


Ну склоняем же :) "Эрика Траутмана"


Is "e.g." not an abbr that we agreed on?


nice one :)


Field "homepage" renamed "homepageUrl" in PluginWSCommons


Haha oops.


Spaces around '+' please.


Thanks for this, Joe.  The #if directives fixed the Arch Linux AUR version when arch moved to Boost 1.58.


Maybe I should use eclipse (PHP editor with syntax check and autocomplete and stuff) instead of NPP...